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

Conversation

@tkt989
Copy link
Contributor

@tkt989 tkt989 commented Feb 2, 2020

概要

docs/add-offline-support-with-a-service を翻訳しました。
レビューよろしくお願いします:bow:

チェックリスト

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

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

補足

Ref: #1

@tkt989 tkt989 changed the title Docs/add offline support with a service worker Translate docs/add offline support with a service worker Feb 2, 2020
@tkt989 tkt989 changed the title Translate docs/add offline support with a service worker Translate docs/add-offline-support-with-a-service-worker Feb 2, 2020
@smorimoto smorimoto added the status: awaiting review A pull request that is currently awaiting a reviewer's response label Feb 2, 2020
hirotaka
hirotaka previously approved these changes Feb 4, 2020
Copy link
Contributor

@hirotaka hirotaka left a comment

Choose a reason for hiding this comment

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

ありがとうございます!とても読みやすいです。

@hirotaka hirotaka 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 4, 2020
Comment on lines 83 to 84
- [Service Worker の紹介](https://developers.google.com/web/fundamentals/primers/service-workers/?hl=ja)
- [サービスワーカー API](https://developer.mozilla.org/ja/docs/Web/API/Service_Worker_API)
Copy link
Contributor

@smorimoto smorimoto Feb 6, 2020

Choose a reason for hiding this comment

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

Service Workerなのか、サービスワーカーなのか統一したいと思ったりしました。 CC: @uetchy @Naturalclar @hirotaka

Copy link
Contributor

Choose a reason for hiding this comment

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

リンク先のMozillaのドキュメントのタイトルがサービスワーカーAPIで、本文内ではService Workerになっていたので、このままでいいのかなと思ったのですが、統一したいですよね。
https://developer.mozilla.org/ja/docs/Web/API/Service_Worker_API

Webの仕様の名称なので、Service Workerのままがいいかなと思うのですが、いかがでしょうか? @uetchy @Naturalclar

Copy link
Contributor

Choose a reason for hiding this comment

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

基本的に記事のタイトルに準じますが、これは規格名なのでService Workerのほうが良いと思います。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

確かに規格名ですので、そのまま書いたほうがいいですね。修正します。
以前、担当したページにも同じリンクがあったので、そちらの方も修正してPRしますね。
#128

@tkt989 tkt989 mentioned this pull request Feb 7, 2020
5 tasks
@Naturalclar Naturalclar merged commit d38c6be into gatsbyjs:master Feb 8, 2020
@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.

5 participants