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

Conversation

@chanrute
Copy link
Contributor

@chanrute chanrute commented Mar 10, 2020

概要

#1 の対応として、 docs/running-queries-with-graphiql を翻訳しました。

チェック一覧

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

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

補足

Ref: #1

@smorimoto smorimoto added the status: awaiting review A pull request that is currently awaiting a reviewer's response label Mar 10, 2020
@smorimoto
Copy link
Contributor

ちゃんとチェック一覧にあることを行ってからチェック一覧を埋めるようにしてください。

@chanrute
Copy link
Contributor Author

@imbsky
了解しました。気をつけます。

"ボイラーテンプレート" was fixed in an unnecessary place.
@uetchy uetchy 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 Mar 14, 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.

@f-teruhisa PR ありがとうございます!
表示されている差分が原文と位置がずれているようなので、一度内容を確認おねがいしたいです。
原文のドキュメントが更新されたときに、変更箇所が日本語でも検知出来るように、できるだけ文章の位置を原文にそろえていただけると助かります。

@chanrute
Copy link
Contributor Author

@Naturalclar @imbsky
ご指摘ありがとうございます。
修正させていただきましたので、お手すきの際にレビュー頂ければ幸いです。

@@ -1,40 +1,40 @@
---
title: GraphQL Concepts
タイトル: GraphQLのコンセプト
Copy link
Contributor

Choose a reason for hiding this comment

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

メタデータのキーは翻訳してはいけません。

uetchy
uetchy previously approved these changes Apr 20, 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 author response A pull request that is awaiting author's response labels Apr 20, 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 merged commit 7dffd91 into gatsbyjs:master May 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants