Skip to content

dev の変更を振り返り機能のブランチに取り込む - #7138

Merged
TinyKitten merged 3 commits into
feature/ride-reviewfrom
chore/ride-review-from-dev
Sep 29, 2026
Merged

TinyKitten merged 3 commits into
feature/ride-reviewfrom
chore/ride-review-from-dev

Conversation

@TinyKitten

Copy link
Copy Markdown
Member

概要

dev の変更を、振り返り機能(#5751)の epic のブランチ feature/ride-review に取り込みます。

マージは「Create a merge commit」で行ってください(squash しないでください)。 squash すると dev の履歴とのつながりが切れ、次に feature/ride-review を dev に入れるときに同じ変更がもう一度衝突します。

変更の種類

  • バグ修正
  • 新機能
  • リファクタリング
  • ドキュメント
  • CI/CD
  • その他

変更内容

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

@TinyKitten TinyKitten self-assigned this Sep 29, 2026
@coderabbitai

coderabbitai Bot commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Essentials

Run ID: 74a5047c-163e-4b7b-a6f2-d05cb27246c8

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant