-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
i18n(ja): Update endpoints.mdx #4518
Conversation
✅ Deploy Preview for astro-docs-2 ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@morinokami 最新の英語版と比較してレビューしました。すべての変更がわかりやすく翻訳された上で反映されていました。個人的な意見になりますが、1つだけコメントをつけています。元のままでも問題ないかと思いますので、反映するかどうか含めてお任せします。
Co-authored-by: Kyosuke Nakamura <kyosuke@users.noreply.github.com>
@kyosuke ありがとうございます!確かに「任意の」よりも「どんな」のほうが誤解もなく硬すぎもせずいいですね(自分の癖で、硬い言葉を選びがちです...)。description の方もあわせて修正しました(こちらは英文をちゃんと確認すると Learn で始まっていたので、もとのままでもいいのですがついでに「説明します」を「学びます」に変えておきました)。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@morinokami 修正ありがとうございます。言葉選び、むずかしいですよね。
Learnに変わっていたの気づいていませんでした。学びますへの修正で良いと思います!
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Great work peeps 🚀
What kind of changes does this PR include?
Description
astro.build/asset
links to live assets in repo #3016endpoints
#4465