Skip to content
This repository was archived by the owner on Jan 20, 2025. It is now read-only.

Conversation

@u00000ka
Copy link
Contributor

@u00000ka u00000ka commented Jan 19, 2020

概要

チュートリアルのpart-eightを翻訳しました。レビューお願いします。
「Audits」の訳を「検証」「検査」などにするか悩んで「評価」にしました。

チェックリスト

  • 翻訳スタイルガイド に目を通しました。
  • Translation Guide に目を通しました。
  • textlint を使って校正を行いました(推奨)
  • Allow edits from maintainers にチェックを入れました。

しばらく待ってもレビューが終わらなかったり、必要なレビュー数が足りない状態が続いた場合は、こちらからメンテナーを探して、@を付けてメンションを飛ばしてください。

@smorimoto smorimoto added status: awaiting review A pull request that is currently awaiting a reviewer's response status: WIP Work in progress labels Jan 20, 2020
@u00000ka u00000ka changed the title WIP: Translate docs/tutorial/eight Translate docs/tutorial/eight Jan 22, 2020
@u00000ka u00000ka marked this pull request as ready for review January 22, 2020 16:40
@Naturalclar Naturalclar added core pages and removed status: WIP Work in progress labels Jan 23, 2020
---

Wow! You've come a long way! You've learned how to:
ああ!長い道のりでしたね!あなたは以下の内容を学びました。
Copy link
Contributor

Choose a reason for hiding this comment

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

[imo] 「あなたの」を外すと翻訳感が消えて良いかも

Suggested change
ああ!長い道のりでしたね!あなたは以下の内容を学びました
お疲れさまでした!ここまでで以下の内容を学びましたね

```

> 💡 As you learned in [part 1](/tutorial/part-one/), this does a production build of your site and outputs the built static files into the `public` directory.
> 💡 [パート 1](/tutorial/part-one/)で学習したように、これによりサイトのプロダクション用ビルドが実行され、ビルドされた静的ファイルが `public` ディレクトリーに出力されます。
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
> 💡 [パート 1](/tutorial/part-one/)で学習したように、これによりサイトのプロダクション用ビルドが実行され、ビルドされた静的ファイルが `public` ディレクトリーに出力されます。
> 💡 [パート 1](/tutorial/part-one/) で学習したように、これによりサイトのプロダクション用ビルドが実行され、ビルドされた静的ファイルが `public` ディレクトリーに出力されます。

```

The above example is based off the [Gatsby Starter Blog](/starters/gatsbyjs/gatsby-starter-blog/). By passing props to the `<SEO>` component, you can dynamically change the metadata for a post. In this case, the blog post `title` and `excerpt` (if it exists in the blog post markdown file) will be used instead of the default `siteMetadata` properties in your `gatsby-config.js` file.
上記の例は [Gatsby スターターブログ](/starters/gatsbyjs/gatsby-starter-blog/) に基づいています。`<SEO>` コンポーネントに props を渡すことで、ブログ記事のメタデータを動的に変更できます。この場合、`siteMetadatagatsby-config.js` ファイルのデフォルトの `siteMetadata` プロパティの代わりに、投稿ブログの `title``excerpt`(ブログ記事のマークダウンファイルに存在する場合)が使用されます。
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
上記の例は [Gatsby スターターブログ](/starters/gatsbyjs/gatsby-starter-blog/) に基づいています。`<SEO>` コンポーネントに props を渡すことで、ブログ記事のメタデータを動的に変更できます。この場合、`siteMetadatagatsby-config.js` ファイルのデフォルトの `siteMetadata` プロパティの代わりに、投稿ブログの `title``excerpt`(ブログ記事のマークダウンファイルに存在する場合)が使用されます。
上記の例は [Gatsby スターターブログ](/starters/gatsbyjs/gatsby-starter-blog/)に基づいています。`<SEO>` コンポーネントに props を渡すことで、ブログ記事のメタデータを動的に変更できます。この場合、`siteMetadatagatsby-config.js` ファイルのデフォルトの `siteMetadata` プロパティの代わりに、投稿ブログの `title``excerpt`(ブログ記事のマークダウンファイルに存在する場合)が使用されます。

さて、この状態で Lighthouse の評価を再度実施すると、完全ではないにしてもスコア 100 に近づくはずです!

> 💡 For further reading and examples, check out [Adding an SEO Component](/docs/add-seo-component/) and the [React Helmet docs](https://github.com/nfl/react-helmet#example)!
> 💡 さらに読むと例については、[SEO コンポーネントの追加](/docs/add-seo-component/)[React Helmet のドキュメント](https://github.com/nfl/react-helmet#example) をご覧ください!
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
> 💡 さらに読むと例については[SEO コンポーネントの追加](/docs/add-seo-component/)[React Helmet のドキュメント](https://github.com/nfl/react-helmet#example) をご覧ください!
> 💡 さらなる例については[SEO コンポーネントの追加](/docs/add-seo-component/)[React Helmet のドキュメント](https://github.com/nfl/react-helmet#example) をご覧ください!

## こちらで以上です、みなさん!

Well, not quite; just for this tutorial. There are [Additional Tutorials](/tutorial/additional-tutorials/) to check out for more guided use cases.
このチュートリアルに限っては、ということです。様々なユースケースを[追加のチュートリアル](/tutorial/additional-tutorials/)で紹介しています。
Copy link
Contributor

@Naturalclar Naturalclar Jan 23, 2020

Choose a reason for hiding this comment

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

この辺の訳はやりづらいですが、ノリをあわせなくても良いかなと感じました。

Suggested change
このチュートリアルに限っては、ということです。様々なユースケースを[追加のチュートリアル](/tutorial/additional-tutorials/)で紹介しています
基本的なチュートリアルはこれで終了です。その他、様々なユースケースを[追加のチュートリアル](/tutorial/additional-tutorials/)で紹介していますので、ぜひご覧ください

Copy link
Contributor

@Naturalclar Naturalclar left a comment

Choose a reason for hiding this comment

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

全体的に良い翻訳と感じました!
細かいニュアンスやスペースの統一を保つための Suggest を行ったので確認お願いします。

@Naturalclar Naturalclar added status: awaiting author response A pull request that is awaiting author's response and removed status: awaiting review A pull request that is currently awaiting a reviewer's response labels Jan 23, 2020
@u00000ka
Copy link
Contributor Author

u00000ka commented Jan 23, 2020

レビューありがとうございます。修正しました。

@smorimoto smorimoto added status: awaiting review A pull request that is currently awaiting a reviewer's response and removed status: awaiting author response A pull request that is awaiting author's response labels Jan 23, 2020
Copy link
Contributor

@uetchy uetchy left a comment

Choose a reason for hiding this comment

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

🚀

Copy link
Contributor

@Naturalclar Naturalclar left a comment

Choose a reason for hiding this comment

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

LGTM! ありがとうございます!

@Naturalclar Naturalclar merged commit bad0aef into gatsbyjs:master Jan 23, 2020
@gatsbot
Copy link

gatsbot bot commented Jan 23, 2020

Holy buckets, @yukapomeranian — we just merged your PR to Gatsby! 💪💜

Gatsby is built by awesome people like you. Let us say “thanks” in two ways:

  1. We’d like to send you some Gatsby swag. As a token of our appreciation, you can go to the Gatsby Swag Store and log in with your GitHub account to get a coupon code good for one free piece of swag. We’ve got Gatsby t-shirts, stickers, hats, scrunchies, and much more. (You can also unlock even more free swag with 5 contributions — wink wink nudge nudge.) See gatsby.dev/swag for details.
  2. We just invited you to join the Gatsby organization on GitHub. This will add you to our team of maintainers. Accept the invite by visiting https://github.com/orgs/gatsbyjs/invitation. By joining the team, you’ll be able to label issues, review pull requests, and merge approved pull requests.

If there’s anything we can do to help, please don’t hesitate to reach out to us: tweet at @gatsbyjs and we’ll come a-runnin’.

Thanks again!

@uetchy uetchy added core pages and removed core pages status: awaiting review A pull request that is currently awaiting a reviewer's response labels Jan 26, 2020
@tkt989 tkt989 mentioned this pull request Jan 28, 2020
5 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants