Skip to content

i18n(ja): Update /guides/deploy/netlify.mdx#5543

Merged
yanthomasdev merged 7 commits intowithastro:mainfrom
kimulaco:i18n-ja-update-guides-deploy-netlify
Dec 3, 2023
Merged

i18n(ja): Update /guides/deploy/netlify.mdx#5543
yanthomasdev merged 7 commits intowithastro:mainfrom
kimulaco:i18n-ja-update-guides-deploy-netlify

Conversation

@kimulaco
Copy link
Contributor

@kimulaco kimulaco commented Dec 2, 2023

Description (required)

I have reflected the following updates to the Japanese page of /guides/deploy/netlify.mdx.

Related issues & labels (optional)

  • Closes #
  • Suggested label:

@vercel
Copy link

vercel bot commented Dec 2, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview Dec 3, 2023 4:42pm

@github-actions github-actions bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label Dec 2, 2023
@ryuapp
Copy link
Contributor

ryuapp commented Dec 3, 2023

@kimulaco
確認しました
#4859 のPRの13, 19, 21行目の修正が反映されていないようです

現状の翻訳部分は問題ないと思います👍

@kimulaco
Copy link
Contributor Author

kimulaco commented Dec 3, 2023

@ryuapp レビューありがとうございます。
失礼しました。更新漏れていた箇所を修正しました。 🙇 03eafb9

上記の修正の中で、現在日本語版では英語版の21行目相当の文章が抜けていたので追加しました。
その際に23行目と重複した表現になってしまっていたため、23行目の文章も調整してみています。 🙏

Copy link
Contributor

@ryuapp ryuapp left a comment

Choose a reason for hiding this comment

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

追加の修正ありがとうございます
何点かコメントいたしましたので、お手すきの際にご確認ください

AstroプロジェクトでSSRを有効にし、Netlifyにデプロイするには、以下の`astro add`コマンドで[Netlifyアダプター](/ja/guides/integrations-guide/netlify/)を追加します。これにより、アダプターがインストールされ、`astro.config.mjs`ファイルが1つのステップで適切に変更されます。
AstroプロジェクトでSSRを有効にし、NetlifyのEdge Functionsを使用してNetlifyにデプロイするには:

以下の`astro add`コマンドで[Netlifyアダプター](/ja/guides/integrations-guide/netlify/)を追加し、AstroプロジェクトでSSRを有効にします。これによりアダプタがインストールされ、`astro.config.mjs`ファイルに適切な変更が1ステップで行われます。
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 add`コマンドで[Netlifyアダプター](/ja/guides/integrations-guide/netlify/)を追加し、AstroプロジェクトでSSRを有効にします。これによりアダプタがインストールされ`astro.config.mjs`ファイルに適切な変更が1ステップで行われます。
以下の`astro add`コマンドで[Netlifyアダプター](/ja/guides/integrations-guide/netlify/)を追加し、AstroプロジェクトでSSRを有効にします。これによりアダプターがインストールされ`astro.config.mjs`ファイルに適切な変更が1ステップで行われます。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ありがとうございます。こちらで修正しました。 564cd5d


[Netlifyの実験的Edge Functions](https://docs.netlify.com/netlify-labs/experimental-features/edge-functions/#app)を使ってプロジェクトをレンダリングするには、Astro設定ファイルの`netlify/functions`インポートを`netlify/edge-functions`に変更します。
```js title="astro.config.mjs" ins={3} del={2}
Netlifyアダプタの設定に`edgeMiddleware: true`を追加することで、NetlifyのEdge Functionsを使用してプロジェクトをデプロイすることもできます。
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
Netlifyアダプタの設定に`edgeMiddleware: true`を追加することで、NetlifyのEdge Functionsを使用してプロジェクトをデプロイすることもできます。
Netlifyアダプターの設定に`edgeMiddleware: true`を追加することで、NetlifyのEdge Functionsを使用してプロジェクトをデプロイすることもできます。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ありがとうございます。こちらも修正しました。 564cd5d

</p>

ビルドアダプターにより、好みのサーバー、サーバーレス、エッジのホストにデプロイできます。[Netlify](/ja/guides/deploy/netlify/#ssredge用のアダプターを追加する)や[Vercel](/ja/guides/deploy/vercel/#ssrアダプター)などのファーストパーティーのアダプターをインポートし、AstroのSSRを活用しましょう。
ビルドアダプターにより、好みのサーバー、サーバーレス、エッジのホストにデプロイできます。[Netlify](/ja/guides/deploy/netlify/#ssr用のアダプターを追加する)や[Vercel](/ja/guides/deploy/vercel/#ssrアダプター)などのファーストパーティーのアダプターをインポートし、AstroのSSRを活用しましょう。
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が更新されてしまい、翻訳の更新履歴が追えなくなってしまうかもしれません
お手数ですが、新しく先頭に[i18nIgnore]を付けてPRを作成して頂ければと思います

上記、私もやったことが無く申し訳ないのですが、下記に翻訳の履歴を更新しないやり方として書かれていますので参考までにご確認ください
https://github.com/withastro/docs/blob/main/contributor-guides/translating-astro-docs.md#pull-request-guidelines

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ありがとうございます。失礼しました 🙇

こちらrevertし ( 0547b75 )、URLの修正を別PRとして作りました。
[i18nIgnore] Update netlify URL in /ja/reference/configuration-reference.mdx #5557

Copy link
Contributor

@ryuapp ryuapp 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 🚀

@kimulaco kimulaco deleted the i18n-ja-update-guides-deploy-netlify branch December 4, 2023 00:22
ematipico pushed a commit that referenced this pull request Jan 26, 2024
* i18n(ja): Update `/guides/deploy/netlify.mdx`

* i18n(ja): Fix missing update of #4859

* i18n(ja): Fix anchor hash

* Revert "i18n(ja): Fix anchor hash"

This reverts commit e4a7d29.

* i18n(ja): Fix to unify words

* Fix link

@tracker-major:./src/content/docs/ja/guides/deploy/netlify.mdx;

---------

Co-authored-by: Yan Thomas <61414485+Yan-Thomas@users.noreply.github.com>
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