Skip to content

設定画面に正式版ではないことを明記#4656

Merged
TinyKitten merged 1 commit intodevfrom
feature/beta-credit
Nov 22, 2025
Merged

設定画面に正式版ではないことを明記#4656
TinyKitten merged 1 commit intodevfrom
feature/beta-credit

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Nov 22, 2025

fixes #4655

Summary by CodeRabbit

  • 新機能
    • アプリ設定画面にベータ版およびカナリア版リリースの通知を追加しました。ユーザーは現在のTrainLCDのリリース状況をアプリ設定で確認できるようになります。複数言語での対応も実施しています。

✏️ Tip: You can customize this high-level summary in your review settings.

@TinyKitten TinyKitten self-assigned this Nov 22, 2025
@github-actions github-actions bot added the react label Nov 22, 2025
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 22, 2025

Note

.coderabbit.yml has unrecognized properties

CodeRabbit is using all valid settings from your configuration. Unrecognized properties (listed below) have been ignored and may indicate typos or deprecated fields that can be removed.

⚠️ Parsing warnings (1)
Validation error: Unrecognized key(s) in object: 'auto_review'
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Walkthrough

翻訳ファイルに「betaNotice」と「canaryNotice」の2つの翻訳キーを英語と日本語で追加し、AppSettingsコンポーネントの下部に新しいTypographyコンポーネントをレンダリングして、デバイスの状態に応じて該当通知を表示するように実装しました。

Changes

Cohort / File(s) Summary
翻訳ファイル更新
assets/translations/en.json, assets/translations/ja.json
betaNoticeとcanaryNoticeの翻訳キーを追加。英語版では"This TrainLCD is currently in beta release."と"This TrainLCD is a canary release."、日本語版では対応する日本語文字列を追加。
アプリ設定画面UI更新
src/screens/AppSettings.tsx
betaNoticeスタイルシートの新規追加とTypographyコンポーネントをレンダリング。isDevAppの値に応じてcanaryNoticeまたはbetaNoticeを表示。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • 翻訳キーの追加はスキーマに準じた単純な変更
  • AppSettingsコンポーネントへの追加ロジックは条件分岐がシンプルで、既存パターンに従っている

Suggested labels

react

Poem

🐰 新しい通知が加わりました、
ベータとカナリア、二つの声で、
アプリの状態を優しく教える。
翻訳も揃った、日本語も英語も、
ユーザーへの思いやり、ここに結実。 ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 設定画面にベータ版・カナリア版の表示を追加する変更を的確に説明しており、変更内容とタイトルが一致しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/beta-credit

📜 Recent review details

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 36bc45d and 4d22f16.

📒 Files selected for processing (3)
  • assets/translations/en.json (1 hunks)
  • assets/translations/ja.json (1 hunks)
  • src/screens/AppSettings.tsx (2 hunks)
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: TinyKitten
Repo: TrainLCD/MobileApp PR: 0
File: :0-0
Timestamp: 2025-08-11T21:13:39.083Z
Learning: TinyKitten prefers concise code reviews for the TrainLCD/MobileApp repository. Keep review comments brief and to the point.
Learnt from: TinyKitten
Repo: TrainLCD/MobileApp PR: 4448
File: functions/src/funcs/tts.ts:214-232
Timestamp: 2025-08-11T20:23:23.071Z
Learning: PR #4448 in TrainLCD/MobileApp is specifically about AI feedback summarization and labeling functionality, not about TTS (Text-to-Speech) improvements. The TTS code changes appear to be part of a modularization refactoring but are not the focus of this PR.
Learnt from: TinyKitten
Repo: TrainLCD/MobileApp PR: 0
File: :0-0
Timestamp: 2024-12-18T00:29:05.817Z
Learning: 翻訳ファイル(`assets/translations/en.json` と `assets/translations/ja.json`)の変更を要約する際、`backgroundPermissionDenied` のようなキーが追加されているのか削除されているのかを正確に確認する。
📚 Learning: 2024-12-18T00:29:05.817Z
Learnt from: TinyKitten
Repo: TrainLCD/MobileApp PR: 0
File: :0-0
Timestamp: 2024-12-18T00:29:05.817Z
Learning: 翻訳ファイル(`assets/translations/en.json` と `assets/translations/ja.json`)の変更を要約する際、`backgroundPermissionDenied` のようなキーが追加されているのか削除されているのかを正確に確認する。

Applied to files:

  • assets/translations/ja.json
  • assets/translations/en.json
🧬 Code graph analysis (1)
src/screens/AppSettings.tsx (3)
src/utils/rfValue.ts (1)
  • RFValue (3-18)
src/utils/isDevApp.ts (1)
  • isDevApp (7-10)
src/translation.ts (1)
  • translate (7-12)
🔇 Additional comments (4)
assets/translations/en.json (1)

196-198: LGTM!

翻訳キーが正しく追加されています。英語テキストも明確で文法的に正確です。

assets/translations/ja.json (1)

197-199: LGTM!

翻訳キーが正しく追加されています。日本語訳も自然で適切です。

src/screens/AppSettings.tsx (2)

59-64: LGTM!

スタイル定義が適切です。フォントサイズ、中央揃え、上部の余白が通知表示に適しています。


295-297: LGTM!

条件分岐のロジックが正しく実装されています。開発版では「canaryNotice」、本番版では「betaNotice」が表示されます。設定画面の下部に配置されているのも適切です。


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

@TinyKitten TinyKitten merged commit 2e23b35 into dev Nov 22, 2025
6 checks passed
@TinyKitten TinyKitten deleted the feature/beta-credit branch November 22, 2025 11:22
TinyKitten added a commit that referenced this pull request Nov 22, 2025
設定画面に正式版ではないことを明記 (#4656)
@coderabbitai coderabbitai bot mentioned this pull request Jan 18, 2026
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