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

Conversation

@HelloTakako
Copy link
Contributor

概要

docs/path-prefixを翻訳しました。レビューをよろしくお願いいたします。

チェックリスト

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

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

補足

Ref: #1

@smorimoto smorimoto self-assigned this Feb 9, 2020
@smorimoto smorimoto added the status: awaiting review A pull request that is currently awaiting a reviewer's response label Feb 9, 2020
@smorimoto smorimoto removed their assignment Feb 9, 2020
const answer = window.confirm(
`アプリケーションが更新されました。 ` +
`最新版を表示しますか?`
`アプリケーションが更新されました。 ` + `最新版を表示しますか?`
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.

こちらなのですが、commitする際のlintで変更されてしまったようで、Discord上で他の方が作業ファイル以外のエラーが出るようになったとコメントされていたので、あまり深く考えずにPRしてしまいました…
ローカルで変更してまたcommitすると自動で改行が消されてしまい、理由が分からなかったので、先ほどGithub上から直接ファイルを変更しました。

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.

翻訳内容は読みやすいと感じました!
スペースの箇所に関して、他の docs と統一させるために細かい箇所を suggest したので確認おねがいします。

@Naturalclar Naturalclar 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 Feb 9, 2020
export default function Index() {
return (
<Layout>
{/* Note: this is an intentionally contrived example, but you get the idea! */}
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.

はい。

「以下の例は、参考の為のコードであって実用的ではありません」というような訳をするべきか迷ったものの、
72行目の
Go to page 2, dynamically
にかかる冗談的な要素が強いのかなと考えて、削除しました。

Copy link
Contributor

Choose a reason for hiding this comment

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

Dynamically というのは ダイナミックに!みたいな意味ではなく、「動的に移動する」と言う意味で冗談で書いてあるわけではないので、その部分は原文に合わせてもらえると助かります:bow:

@HelloTakako
Copy link
Contributor Author

@Naturalclar さん・@uetchy さん
レビューありがとうございます。
ご指摘頂いた部分を修正・返信いたしましたので、
お手数なのですがご確認をよろしくお願いいたします。


The [`assetPrefix`](/docs/asset-prefix/) feature can be thought of as semi-related to this feature. That feature allows your assets (non-HTML files, e.g. images, JavaScript, etc.) to be hosted on a separate domain, for example a CDN.
[`assetPrefix`](/docs/asset-prefix/) は、[`withPrefix`](/docs/gatsby-link/#add-the-path-prefix-to-paths-using-withprefix) に関連する機能です。
[`assetPrefix`](/docs/asset-prefix/) を使えば、CDN など別ドメインにホスティングされた HTML 以外のファイル(例:画像ファイルや JavaScript ファイルなど)へのパスを自動で追加できます。
Copy link
Contributor

Choose a reason for hiding this comment

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

1行にまとめて、改行を原文と合わせるようにお願いします:bow: (英語の文章が更新された時、そのDiffがきれいに日本語版にも反映できるようにするためです。)

Suggested change
[`assetPrefix`](/docs/asset-prefix/) を使えば、CDN など別ドメインにホスティングされた HTML 以外のファイル(例:画像ファイルや JavaScript ファイルなど)へのパスを自動で追加できます。
この機能を使えば、CDN など別ドメインにホスティングされた HTML 以外のファイル(例:画像ファイルや JavaScript ファイルなど)へのパスを自動で追加できます。

@HelloTakako
Copy link
Contributor Author

@Naturalclar さん @uetchy さん
大変申し訳ありません。
Gitの操作を誤って、大量のファイルを上げてしまいました。
一度このPRを閉じて、また新しくPRを作成した方が良いでしょうか?

@uetchy
Copy link
Contributor

uetchy commented Feb 11, 2020

interactive rebaseを使えばヒストリーを綺麗にできますよ。難しければこちらでやるので教えてください👍

@HelloTakako
Copy link
Contributor Author

@uetchy さん
ごめんなさい、やり方が分からないので、よろしくお願いいたします。

@uetchy
Copy link
Contributor

uetchy commented Feb 11, 2020

ヒストリーを掃除しました🧼

@HelloTakako
Copy link
Contributor Author

ありがとうございます!

@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 Feb 12, 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.

ありがとうございます!

@Naturalclar Naturalclar merged commit 614f7b2 into gatsbyjs:master Feb 12, 2020
@uetchy uetchy removed the status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned label Feb 16, 2020
@HelloTakako HelloTakako deleted the docs/path-prefix branch February 17, 2020 23:10
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