Repository navigation
dev の変更を振り返り機能のブランチに取り込む - #7138
Merged
Merged
Conversation
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…-dev # Conflicts: # src/screens/AppSettings.tsx
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Essentials Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
This was referenced Sep 29, 2026
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
devの変更を、振り返り機能(#5751)の epic のブランチfeature/ride-reviewに取り込みます。変更の種類
変更内容
devにあってfeature/ride-reviewに無かった次の変更を取り込みます。衝突の解き方
src/screens/AppSettings.tsxが衝突しました。dev側(設定画面のパーソナライズの項目を見出しで分ける #7136)は、振り返りの項目が無い前提で見出しを分けていました。feature/ride-review側は、見出しを分ける前の一覧に振り返りの項目を足していました。dev側の見出しの構成を採り、設定画面の「パーソナライズ」の項目を見出しで分ける #7134 で決めた分け方どおり「通知とアナウンス」と「デバイス」のあいだに「アクティビティ」の見出しを足して、振り返りの項目を置きました。見出しの翻訳キーsettingsSectionActivity(アクティビティ / Activity)を足しています。feature/ride-review側にあった、表示言語の行を切り抜く個別の処理は、dev側でステップごとのswitchにまとめられていたので、dev側を採りました。dev側で足されたAppSettings.test.tsxの、見出しと項目の並びを固定するテストに、アクティビティの見出しと振り返りの項目の2行を足しました。見出しで分けること、項目の無い見出しを出さないことの確認は変えていません。テスト
npm run lintが通ることnpm testが通ることnpm run typecheckが通ることマージ後の先頭で
npm run lint、npm run typecheck、npm test(316 スイート・3538 件)を実行し、すべて通りました。実機では確かめていません。関連Issue
Refs #5751
Refs #7134
スクリーンショット(任意)
未添付: 取り込んだ変更の画面は #7136 にあります。このPRで足した見た目の変化は、設定画面に「アクティビティ」の見出しが増え、その下に振り返りの項目が移ることだけで、実機では撮っていません。
🤖 Generated with Claude Code