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

i18n(ja): Update configuration-reference.mdx #4546

Merged

Conversation

morinokami
Copy link
Contributor

What kind of changes does this PR include?

  • New or updated content
  • Translated content

Description

  • Update configuration-reference.mdx.

@netlify
Copy link

netlify bot commented Sep 5, 2023

Deploy Preview for astro-docs-2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit d84d062
🔍 Latest deploy log https://app.netlify.com/sites/astro-docs-2/deploys/64f76314c01f94000803e6f8
😎 Deploy Preview https://deploy-preview-4546--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions github-actions bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label Sep 5, 2023
@morinokami

This comment was marked as resolved.

{/* TODO: 遷移先の翻訳が古く、英語版にあるURLフラグメント(#using-glob-imports)を使えないため、これを削除して対応した。翻訳が完了したら置き換える。 */}
- [AstroにおけるRSSアイテムの生成](/ja/guides/rss/)
- [AstroにおけるRSSアイテムの生成](/ja/guides/rss/#globインポートの使用)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

ここは関係ないですが見つけたのでついでに更新しました。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

新しい訳の追加に加えて、英単語の周りのスペースの除去や既にある訳の改善などもおこなっています。

@morinokami morinokami marked this pull request as ready for review September 5, 2023 12:55
@@ -83,7 +83,7 @@ SSRを有効にしたプロジェクトをデプロイするには、アダプ
</Fragment>
</PackageManagerTabs>

2. `astro.config.mjs`ファイルのimportとdefault exportに[アダプターを追加します](/ja/reference/configuration-reference/#アダプター)。
2. `astro.config.mjs`ファイルのimportとdefault exportに[アダプターを追加します](/ja/reference/configuration-reference/)。
Copy link
Contributor

Choose a reason for hiding this comment

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

ここをそのままで良いのでは?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ありがとうございます。ここをそのままとすると、今回の変更により /ja/reference/configuration-reference/ から「アダプター」の見出しがなくなったので、CI でリンク先をチェックするところでエラーとなってしまうんですよね(https://github.com/withastro/docs/actions/runs/6080484539/job/16494514704 を参照)。一番良い対応は古いリンクが残ってしまっている server-side-rendering.mdx も同時に更新することなのですが、ちょっと量が多そうで後回しにしたいため、苦肉の策としてフラグメントを外してエラーにならないようにしています。

Copy link
Contributor

Choose a reason for hiding this comment

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

あ、configuration-reference.mdx に変更があったの見逃してました 🙇
となると、#adapter が正しいのでは?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

確かに、もとのセクションが消えたわけではなく #adapter に移動しているので、それが良さそうですね!その通りに更新しておきます🙏

<Since v="2.10.10" />
</p>

リモート画像の最適化を許可する画像ソースドメインのリストを定義します。Astroは、その他のリモート画像を最適化しません。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
リモート画像の最適化を許可する画像ソースドメインのリストを定義します。Astroは、その他のリモート画像を最適化しません。
ローカル画像の最適化を許可する画像ソースドメインのリストを定義します。Astroは、その他のリモート画像を最適化しません。

Defines a list of permitted image source domains for local image optimization. No other remote images will be optimized by Astro.

<Since v="2.10.10" />
</p>

リモート画像の最適化を許可する画像ソースURLパターンのリストを定義します。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
リモート画像の最適化を許可する画像ソースURLパターンのリストを定義します。
リモート画像の最適化を許可する画像ソースURLパターンのリストを定義します。

Defines a list of permitted image source URL patterns for local image optimization.

<Since v="2.1.0" />
</p>

どの画像サービスをAstroのアセットサポートで使用するかを設定します。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
どの画像サービスをAstroのアセットサポートで使用するかを設定します
どの画像サービスをAstroの実験的アセットサポートで使用するかを設定します

Astro’s experimental assets support

@morinokami
Copy link
Contributor Author

morinokami commented Sep 5, 2023

@pilcrowonpaper Suggest してもらったうち取り込んでいないもの(「リモート」を「ローカル」に直すもの、「実験的」を追加するもの)について、これはもとの英文が誤りで、withastro/astro#8410 にて直したのですが、ドキュメントサイトに反映されるまでに時間が掛かっていたようです(ちょうどレビューしてもらったタイミングと入れ違いで #4548 にて反映されています)。ので、これらについてはこのままとさせてください!

Copy link
Contributor

@pilcrowonpaper pilcrowonpaper left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Member

@yanthomasdev yanthomasdev 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 peeps 🖖

@yanthomasdev yanthomasdev merged commit eaddc9c into withastro:main Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants