-
-
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 configuration-reference.mdx #4546
i18n(ja): Update configuration-reference.mdx #4546
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. |
This comment was marked as resolved.
This comment was marked as resolved.
{/* TODO: 遷移先の翻訳が古く、英語版にあるURLフラグメント(#using-glob-imports)を使えないため、これを削除して対応した。翻訳が完了したら置き換える。 */} | ||
- [AstroにおけるRSSアイテムの生成](/ja/guides/rss/) | ||
- [AstroにおけるRSSアイテムの生成](/ja/guides/rss/#globインポートの使用) |
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.
ここは関係ないですが見つけたのでついでに更新しました。
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.
新しい訳の追加に加えて、英単語の周りのスペースの除去や既にある訳の改善などもおこなっています。
@@ -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/)。 |
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.
ここをそのままで良いのでは?
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.
ありがとうございます。ここをそのままとすると、今回の変更により /ja/reference/configuration-reference/ から「アダプター」の見出しがなくなったので、CI でリンク先をチェックするところでエラーとなってしまうんですよね(https://github.com/withastro/docs/actions/runs/6080484539/job/16494514704 を参照)。一番良い対応は古いリンクが残ってしまっている server-side-rendering.mdx も同時に更新することなのですが、ちょっと量が多そうで後回しにしたいため、苦肉の策としてフラグメントを外してエラーにならないようにしています。
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.
あ、configuration-reference.mdx
に変更があったの見逃してました 🙇
となると、#adapter
が正しいのでは?
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.
確かに、もとのセクションが消えたわけではなく #adapter
に移動しているので、それが良さそうですね!その通りに更新しておきます🙏
<Since v="2.10.10" /> | ||
</p> | ||
|
||
リモート画像の最適化を許可する画像ソースドメインのリストを定義します。Astroは、その他のリモート画像を最適化しません。 |
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.
リモート画像の最適化を許可する画像ソースドメインのリストを定義します。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パターンのリストを定義します。 |
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.
リモート画像の最適化を許可する画像ソースURLパターンのリストを定義します。 | |
リモート画像の最適化を許可する画像ソースURLパターンのリストを定義します。 |
Defines a list of permitted image source URL patterns for local image optimization.
<Since v="2.1.0" /> | ||
</p> | ||
|
||
どの画像サービスをAstroのアセットサポートで使用するかを設定します。 |
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.
どの画像サービスをAstroのアセットサポートで使用するかを設定します。 | |
どの画像サービスをAstroの実験的アセットサポートで使用するかを設定します。 |
Astro’s experimental assets support
Co-authored-by: pilcrowOnPaper <[email protected]>
Co-authored-by: pilcrowOnPaper <[email protected]>
Co-authored-by: pilcrowOnPaper <[email protected]>
Co-authored-by: pilcrowOnPaper <[email protected]>
Co-authored-by: pilcrowOnPaper <[email protected]>
@pilcrowonpaper Suggest してもらったうち取り込んでいないもの(「リモート」を「ローカル」に直すもの、「実験的」を追加するもの)について、これはもとの英文が誤りで、withastro/astro#8410 にて直したのですが、ドキュメントサイトに反映されるまでに時間が掛かっていたようです(ちょうどレビューしてもらったタイミングと入れ違いで #4548 にて反映されています)。ので、これらについてはこのままとさせてください! |
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 👍
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! Thanks peeps 🖖
What kind of changes does this PR include?
Description