Skip to content

i18n(ja): add guides/integrations-guide/sitemap.mdx - #14779

Open
awwa wants to merge 6 commits into
withastro:mainfrom
awwa:i18n-ja-guides-integrations-guide-sitemap
Open

awwa wants to merge 6 commits into
withastro:mainfrom
awwa:i18n-ja-guides-integrations-guide-sitemap

Conversation

@awwa

@awwa awwa commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

Description

This PR adds the Japanese translation of the sitemap integration guide.

I'd appreciate it if you could review this when you have time 🫡

@github-actions github-actions Bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label Oct 24, 2026
@astrobot-houston

astrobot-houston commented Oct 24, 2026 •

Copy link
Copy Markdown
Contributor

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the tracking.ignoredKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
ja/guides/integrations-guide/sitemap.mdx Localization added, will be marked as complete.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@github-actions

github-actions Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

Preview deployment

✅ Deployment complete!

@awwa
awwa marked this pull request as ready for review October 24, 2026 16:22

<p>

**型:** `<code><a href="#linkitem">LinkItem</a>[] | undefined</code>`

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

<code><a href="#linkitem">LinkItem</a>[] | undefined</code>

ここなのですが、リンクが機能しなくなってるので、code要素前後のバッククォート除去してください〜!

@awwa awwa Oct 25, 2026 •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

バッククォートを削除しました。
バッククォートを削除しただけだと、[]の直後にスペースがあるせいでlint:jaがエラーになります。
このスペースを削除すればlint:jaは通りますが、オリジナルのコンテンツを修正してしまうのはあまり良くない修正方法だと思うので、一旦このままにしておきます。

% pnpm run lint:ja
$ textlint --cache --config .textlintrc.ja.json src/content/docs/ja/**/*.{md,mdx}

~/github/astro/docs/src/content/docs/ja/guides/integrations-guide/sitemap.mdx
  735:48  ✓ error  かっこの外側、内側ともにスペースを入れません。  ja-spacing/ja-no-space-around-parentheses

✖ 1 problem (1 error, 0 warnings)
✓ 1 fixable problem.
Try to run: $ textlint --fix [file]

[ELIFECYCLE] Command failed with exit code 1.

@Yusaku01 Yusaku01 Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@awwa
あー、表現とlintルールの問題ですね...😅
すみません、そうしたら二転三転して申し訳ないのですが、元のルールと競合していなかったバッククォート有り表現に戻していただいて大丈夫です🙏

過去のやり方に基づき、lintの問題は別で作業する方向にしましょう

#14782 で例外処理を入れたので、恐らく最新のupstream/mainをmerge or rebaseでこのPRに取り込んでいただければCIエラー直るかな?と思ってます〜

お手隙で試していただければ🙏

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@Yusaku01
早速のご対応ありがとうございました!
upstream/mainをmergeしたところ通りましたー


ページ数の多い大規模なサイトでは、サイトマップファイルの使用が推奨されます。サイトマップを使用しなくても、ほとんどの検索エンジンはサイトのページを検出できますが、サイトマップを用意することで、サイトを検索エンジンにとって可能な限り扱いやすい状態に保つことができます。

Astro Sitemapを使用すれば、このXMLファイルを手動で作成する必要はありません。Astro Sitemapインテグレーションが静的生成されたルートをクロールし、`getStaticPaths()`によって生成された`[...slug]`や`src/pages/[lang]/[version]/info.astro`などの[動的ルート](/ja/guides/routing/#静的ルーティング)も含めてサイトマップファイルを自動生成します。

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

動的ルート

ここなのですが、挿入されているアンカーが、静的ルーティングになっているので、
#動的ルーティングに修正お願いします〜

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

#動的ルーティングに修正しましたー

@Yusaku01 Yusaku01 left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@awwa
PRの作成ありがとうございます!

2点ほどコメントさせていただきました!
お手隙にご確認お願いします〜🙏

@Yusaku01

Copy link
Copy Markdown
Contributor

@awwa

何度もコメントすみません🙏

#14782 で例外処理を入れたので、恐らく最新のupstream/mainをmerge or rebaseでこのPRに取り込んでいただければCIエラー直るかな?と思ってます〜

お手隙で試していただけるとありがたいです〜🙋‍♂️

@Yusaku01 Yusaku01 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for taking care of this! LGTM!

This branch has not been deployed

No deployments
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