Skip to content

06-property-bindingの和訳を作成 - #1200

Open
darkss-xyzzy wants to merge 2 commits into
angular:mainfrom
darkss-xyzzy:feature/tutorial-first-app-06
Open

06-property-bindingの和訳を作成#1200
darkss-xyzzy wants to merge 2 commits into
angular:mainfrom
darkss-xyzzy:feature/tutorial-first-app-06

Conversation

@darkss-xyzzy

Copy link
Copy Markdown
Contributor

翻訳・修正

変更内容はCONTRIBUTING.md に記載されたワークフローに従っていることを確認してください。

関連Issue

#1198

備考

他のドキュメントと統一されているかを確認お願いします。
また、特にInput周りの説明の理解が合っているか重点的に確認してもらえると助かります。

@darkss-xyzzy

darkss-xyzzy commented Aug 13, 2026

Copy link
Copy Markdown
ContributorAuthor

@lacolacoさん

ciで以下のエラーが出てますね。何でしょうか。ローカルのビルド環境は普通に通っていました。
ERROR: /home/runner/work/angular-ja/angular-ja/build/adev/BUILD.bazel:163:15: JsRunBinary adev/dist failed: (Terminated): _build.production.ng_cli failed: error executing JsRunBinary command (from target //adev:build.production)

追記:
エラーの内容を調べたところ、今回のコミット内容そのものが直接の原因ではなく、
.github/workflows/ci.ymlのビルド時のタイムアウトの値(timeout-minutes:)が引っかかったようです。
こちらの判断で勝手に直すわけにはいきませんので、直していただくことは可能でしょうか。

@darkss-xyzzy

Copy link
Copy Markdown
ContributorAuthor

@lacolaco さん

ciのエラーが解消しています。リカバリしていただいたのでしょうか。
後はレビューが残っていますので、お手数ですが、
お時間ある時にお願いします。

@lacolacolacolaco left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

翻訳ありがとうございます!1点だけ修正をお願いします。

コンポーネントタグにプロパティバインディングを追加する際には、`[attribute] = "value"`という構文を使います。これは割り当てた値を文字列ではなくコンポーネントクラスのプロパティとして扱うようAngularに指示するためのものです。

The value on the right-hand side is the name of the property from the `Home`.
右辺の値は`Home`コンポーネントのプロパティの名です。

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
右辺の値は`Home`コンポーネントのプロパティの名です
右辺の値は`Home`コンポーネントのプロパティ名です

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

@lacolaco さん

レビューありがとうございます。
コンポーネントのプロパティの名です=>コンポーネントのプロパティ名です
に修正しました。

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@darkss-xyzzy@lacolaco