Skip to content

shortCriticalTextに状態プレフィックスを追加して次は・まもなく・通過中を表示#5397

Merged
TinyKitten merged 1 commit intodevfrom
feature/update-short-critical-text
Feb 21, 2026
Merged

shortCriticalTextに状態プレフィックスを追加して次は・まもなく・通過中を表示#5397
TinyKitten merged 1 commit intodevfrom
feature/update-short-critical-text

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Feb 21, 2026

Summary by CodeRabbit

バグ修正

  • 列車運行情報通知のメッセージが端末の言語設定に対応するようになりました。通過駅、接近駅、次駅に関する通知の表示が適切にローカライズされます。

@TinyKitten TinyKitten self-assigned this Feb 21, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 21, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

LiveUpdateModule.ktのshortCriticalTextハンドリングを修正し、分岐の値をリソース文字列ルックアップでラップしてメッセージをローカライズします。駅名参照をgetString()呼び出しで包むように変更。

Changes

Cohort / File(s) Summary
Localization in Notification Text
android/app/src/main/java/me/tinykitten/trainlcd/LiveUpdateModule.kt
shortCriticalText生成時に、passingStationName、nextStationNameなどの駅名値をリソース文字列ルックアップ(live_update_passinglive_update_approachinglive_update_next)でラップしてローカライズするように修正。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested labels

android

Poem

🐰 駅名の文字、世界の言葉に
リソース文字列で優しく装う
メッセージ届け、多言語の景色
小さな修正、大きな翼
ローカライズの輝き、あちこちに✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PRタイトルは、shortCriticalTextに状態プレフィックス(次は・まもなく・通過中)を追加する変更内容を正確に反映しており、変更セット全体の主要な目的を明確に要約しています。

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/update-short-critical-text

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

@TinyKitten TinyKitten merged commit f495f1e into dev Feb 21, 2026
5 checks passed
@TinyKitten TinyKitten deleted the feature/update-short-critical-text branch February 21, 2026 03:46
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