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

Conversation

@2ur1
Copy link
Contributor

@2ur1 2ur1 commented Jan 17, 2020

概要

docs/static-vs-normal-queries の翻訳です。

原文ページ
Static vs Normal Queries

チェックリスト

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

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

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

@smorimoto smorimoto added the status: awaiting review A pull request that is currently awaiting a reviewer's response label Jan 18, 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.

ありがとうございます!
わたしがまだ消化しきれていないのですが、取り急ぎ気付いた箇所だけSuggestionあげさせていただきます。

```

In either case, you can add traced SVG support by adding `_tracedSVG` to the end of each fragment. _Note this won’t work in the GraphQL explorer._
In either case, you can add traced SVG support by adding `_tracedSVG` to the end of each fragment. _注: this won’t work in the GraphQL explorer._
Copy link
Contributor

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.

ありがとうございます。別の翻訳のページで開いていたファイルのタブで編集しちゃったみたいです
ご指摘ありがとうございます。

@hirotaka hirotaka added the status: awaiting author response A pull request that is awaiting author's response label Jan 20, 2020
uetchy
uetchy previously approved these changes Jan 23, 2020
@uetchy
Copy link
Contributor

uetchy commented Jan 23, 2020

@hirotaka 再レビューお願いします

@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 status: awaiting review A pull request that is currently awaiting a reviewer's response labels Jan 23, 2020
@uetchy uetchy requested a review from hirotaka January 23, 2020 06:32
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.

一つだけ細かいのですが、Suggestionあげたのでご確認お願いいたします。

@hirotaka hirotaka added the status: awaiting author response A pull request that is awaiting author's response label Jan 23, 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.

全体的に良いと感じました!1点 Suggest したので確認おねがいします。

@Naturalclar Naturalclar removed the status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned label Jan 26, 2020
@2ur1
Copy link
Contributor Author

2ur1 commented Jan 26, 2020

@Naturalclar
ありがとうございます!修正コミットPUSHしましたのでご確認お願いいたします

@uetchy uetchy 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 27, 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 review A pull request that is currently awaiting a reviewer's response labels Jan 27, 2020
@uetchy uetchy removed the status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned label Jan 27, 2020
@uetchy uetchy added status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned status: ready to merge A pull request that can be merged once CI is green and removed status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned labels Jan 27, 2020
@uetchy
Copy link
Contributor

uetchy commented Jan 27, 2020

@hirotaka 再レビューお願いします

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.

LGTM 👍

@hirotaka hirotaka merged commit c77f381 into gatsbyjs:master Jan 27, 2020
@uetchy uetchy removed the status: ready to merge A pull request that can be merged once CI is green label Jan 27, 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