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

Conversation

@osamumu21
Copy link
Contributor

@osamumu21 osamumu21 commented Feb 2, 2020

概要

docs/using-gatsby-without-graphqlを翻訳致しました。
内容のご確認とご指導のほど宜しくお願い申し上げます。

チェックリスト

  • 翻訳スタイルガイド に目を通しました。
  • Translation Guide に目を通しました。
  • textlint を使って校正を行いました。
  • 文章全体を最初から読み直して不自然な箇所が無いことを確認しました。
  • Allow edits from maintainers にチェックを入れました。

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

補足

“unstructured data”を直訳で訳しましたが不自然であれば直したいと存じます。

Ref: #1

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.

PR ありがとうございます!いくつか Suggest したのでご確認お願いします。

@Naturalclar Naturalclar added the status: awaiting author response A pull request that is awaiting author's response label Feb 3, 2020
@osamumu21
Copy link
Contributor Author

早速のレビューありがとうございました!
ご指摘いただいたところ(と気づいたところ)修正致しましたので再度ご確認のほど宜しくお願い申し上げます。

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 しました。
基本的に以下の方針で統一しています。

  • 半角文字と全角文字の間はスペースを開ける(Linter がほとんどはやってくれますが、リンクで囲まれたりしていると、Linter が検知しない部分があります)
  • 全角文字と全角文字の間はスペースを開けない(同じく Linter がチェックしていますが、間にリンクがあると、その前の文字が全角だったかどうかを Linter が検知しない)
  • 例外として、句読点の後はスペースを入れない。

@osamumu21
Copy link
Contributor Author

たくさんのご指摘ありがとうございます!
理解度が足りずミスが多くすみませんでした。
スペースのルールに気をつけて全体を修正いたしました。
レビューのほど宜しくお願い申し上げます。

@osamumu21 osamumu21 requested a review from Naturalclar February 4, 2020 15:27
Naturalclar
Naturalclar previously approved these changes Feb 4, 2020
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 added status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned and removed status: awaiting author response A pull request that is awaiting author's response labels Feb 4, 2020
## Gatsby にとっておすすめは?

If you're building a small site, one efficient way to build it is to pull in unstructured data as outlined in this guide, using `createPages` API, and then if the site becomes more complex later on, you move on to building more complex sites, or you'd like to transform your data, follow these steps:
もしあなたが作ろうとしているサイトの規模が小さいものでしたら、このガイドで説明したように `createPages` API を使って「非構造データ」を取り込むことが効率的でしょう。あとでもしサイトが大きくなったり、より複雑なサイトの構築やデータを加工したくなった場合は下記のような手順を実施ください。
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
もしあなたが作ろうとしているサイトの規模が小さいものでしたら、このガイドで説明したように `createPages` API を使って「非構造データ」を取り込むことが効率的でしょう。あとでもしサイトが大きくなったり、より複雑なサイトの構築やデータを加工したくなった場合は下記のような手順を実施ください
もしあなたが作ろうとしているサイトの規模が小さいものでしたら、このガイドで説明したように `createPages` API を使って「非構造データ」を取り込むことが効率的でしょう。あとでもしサイトが大きくなったり、より複雑なサイトの構築やデータを加工したくなった場合は下記のような手順を実施してください

@osamumu21
Copy link
Contributor Author

レビューありがとうございました!
ご指摘いただいたところ修正致しましたので再度ご確認のほど宜しくお願い申し上げます。

@smorimoto smorimoto added status: awaiting review A pull request that is currently awaiting a reviewer's response and removed status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned labels Feb 6, 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.

🚀

@uetchy uetchy added status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned and removed status: awaiting review A pull request that is currently awaiting a reviewer's response labels Feb 6, 2020
@Naturalclar Naturalclar merged commit 4afb33c into gatsbyjs:master Feb 8, 2020
@gatsbot
Copy link

gatsbot bot commented Feb 8, 2020

Holy buckets, @osamumu21 — 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 removed the status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned label Feb 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants