Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(macros/LearnSidebar): add Japanese translation for 4 locations #11351

Merged
merged 1 commit into from
Jun 25, 2024

Conversation

gurezo
Copy link
Contributor

@gurezo gurezo commented Jun 22, 2024

Summary

@hmatrjp @potappo , mdn-yari-ja

please review for this pull request

Problem

still en-us only

English pages

Japanese pages

  • パフォーマンス — ウェブサイトを高速かつ応答性の高いものにする
  • パフォーマンスガイド
  • その他のリソース
  • よくある質問
English pages Japanese pages
スクリーンショット 2024-06-22 13 32 04 スクリーンショット 2024-06-22 13 40 13
スクリーンショット 2024-06-22 13 31 49 スクリーンショット 2024-06-22 13 40 45

Solution

translate japanese for LearnSidebar.ejs

NOTICE

@caugner, @wbamberg , @schalkneethling

Please merge once mdn-yari-ja is approved.

Before

still en-us only

After

add tranlate japanese for LearnSidebar.ejs

@gurezo gurezo requested a review from a team as a code owner June 22, 2024 04:46
@github-actions github-actions bot added the macros tracking issues related to kumascript macros label Jun 22, 2024
Copy link
Contributor

@potappo potappo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Thanks!

@gurezo
Copy link
Contributor Author

gurezo commented Jun 22, 2024

@caugner, @wbamberg , @schalkneethling

already approved by mdn-yari-ja (@potappo)
please this PR review

@caugner caugner merged commit acecc93 into mdn:main Jun 25, 2024
9 checks passed
@caugner
Copy link
Contributor

caugner commented Jun 25, 2024

Thanks @gurezo!

PS: In the future, I recommend to mention @mdn/core-dev only, because a review from this team is required for the merge.

@gurezo
Copy link
Contributor Author

gurezo commented Jun 25, 2024

Hi @caugner
thanks for reprely, ok next time i will send @mdn/core-dev team!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
macros tracking issues related to kumascript macros
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants