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

Conversation

@hiramatsutaku
Copy link
Contributor

レビューよろしくお願いします 🙇

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.

👍

Suggested changes が妥当だと感じたら、そのコメントに対して(👍)リアクションをしてください。他のレビュワーが変更をコミットします。不適切だと感じたら、単にコメントに対して返事を書いてください。

return setUser({
username: `john`,
name: `Johnny`,
username: `ジョン`,
Copy link
Contributor

Choose a reason for hiding this comment

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

このusernameはGitHubでいうIDのようなものなので、johnのままにしましょう。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ご指摘ありがとうございます 🙇 修正します!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

3efe1d8 にて修正しました 🙇

これで、ログインとユーザー制限エリアを持った認証ワークフローができました!

## Further reading
## 参考文献
Copy link
Contributor

Choose a reason for hiding this comment

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

Further readingなので"関連項目"や"関連する記事"が近いかも?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

こちら同意です 🙇
関連する記事 に修正しますー

Copy link
Contributor Author

Choose a reason for hiding this comment

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

6341600 にて修正しました。

- [景品や Gatsby グッズの Gatsby ストア](https://github.com/gatsbyjs/store.gatsbyjs.org)
- [GatsbyReactWebtask.io でブログを構築しよう!](https://auth0.com/blog/building-a-blog-with-gatsby-react-and-webtask/)
- [JAMstack PWA — Gatsby.jsFirebaseStyled-components でポーリングアプリを構築しよう パート 2](https://medium.com/@UnicornAgency/jamstack-pwa-lets-build-a-polling-app-with-gatsby-js-firebase-and-styled-components-pt-2-9044534ea6bc)
- [JAMstack Hackathon Starter - Netlify Identity を使用した認証済み Gatsby アプリスターター](/starters/sw-yx/jamstack-hackathon-starter)
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
- [JAMstack Hackathon Starter - Netlify Identity を使用した認証済み Gatsby アプリスターター](/starters/sw-yx/jamstack-hackathon-starter)
- [JAMstack Hackathon Starter - Netlify Identity を使用した認証機能付き Gatsby アプリスターター](/starters/sw-yx/jamstack-hackathon-starter)

- React Context API を使用して認証処理を行う [Gatsby メール _アプリケーション_](https://github.com/DSchau/gatsby-mail)
- [景品や Gatsby グッズの Gatsby ストア](https://github.com/gatsbyjs/store.gatsbyjs.org)
- [GatsbyReactWebtask.io でブログを構築しよう!](https://auth0.com/blog/building-a-blog-with-gatsby-react-and-webtask/)
- [JAMstack PWA — Gatsby.jsFirebaseStyled-components でポーリングアプリを構築しよう パート 2](https://medium.com/@UnicornAgency/jamstack-pwa-lets-build-a-polling-app-with-gatsby-js-firebase-and-styled-components-pt-2-9044534ea6bc)
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
- [JAMstack PWA — Gatsby.js、Firebase、Styled-components でポーリングアプリを構築しよう パート 2](https://medium.com/@UnicornAgency/jamstack-pwa-lets-build-a-polling-app-with-gatsby-js-firebase-and-styled-components-pt-2-9044534ea6bc)
- [JAMstack PWA — Gatsby.js、Firebase、Styled-components で投票アプリをつくろう パート 2](https://medium.com/@UnicornAgency/jamstack-pwa-lets-build-a-polling-app-with-gatsby-js-firebase-and-styled-components-pt-2-9044534ea6bc)

---

Sometimes, you need to create a site with gated content, restricted to only authenticated users. Using Gatsby, you may achieve this using the concept of [client-only routes](/docs/client-only-routes-and-user-authentication/), to define which pages a user can view only after logging in.
往々にして、認証済みユーザーのみ閲覧可能なコンテンツを含むサイトを構築しないといけないことがあります。Gatsby を使用すると、[クライアントサイドルーティング](/docs/client-only-routes-and-user-authentication/)を用いて、ユーザーがログイン後にのみ閲覧できるページを作成できます。
Copy link
Contributor

Choose a reason for hiding this comment

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

IMHO

Suggested change
往々にして、認証済みユーザーのみ閲覧可能なコンテンツを含むサイトを構築しないといけないことがあります。Gatsby を使用すると[クライアントサイドルーティング](/docs/client-only-routes-and-user-authentication/)を用いて、ユーザーがログイン後にのみ閲覧できるページを作成できます
往々にして、認証されたユーザーのみが閲覧可能なコンテンツを含むサイトを構築しないといけないことがあります。Gatsby なら[クライアントサイドルーティング](/docs/client-only-routes-and-user-authentication/)を用いてユーザーがログイン後にのみ閲覧できるページを作成できます

uetchy
uetchy previously approved these changes Jan 15, 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.

LGTM🎉(他のレビュワーをお待ちください)

## 前提条件

You should have already configured your environment to be able to use the `gatsby-cli`. A good starting point is the [main tutorial](/tutorial).
事前に `gatsby-cli` を使用できるように環境を設定しておく必要があります。適切な出発点は[メインチュートリアル](/tutorial)です。
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-cli` を使用できるように環境を設定しておく必要があります。適切な出発点は[メインチュートリアル](/tutorial)です
事前に `gatsby-cli` を使用できるように環境を設定しておく必要があります。設定の仕方に関しては、[メインチュートリアル](/tutorial)を参照しましょう

Copy link
Contributor Author

Choose a reason for hiding this comment

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

レビューありがとうございます 🙇 f1b2478 にて修正しました!

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! 👍

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 merged commit 898f95c into gatsbyjs:master Jan 15, 2020
@gatsbot
Copy link

gatsbot bot commented Jan 15, 2020

Holy buckets, @hiramatsutaku — 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!

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.

3 participants