Skip to content

#6614 認証キー設定画面のモーダル内メッセージを多言語対応#6618

Merged
dotani1111 merged 1 commit intoEC-CUBE:4.3from
to19960425:fix/i18n-captcha-modal-message
Mar 5, 2026
Merged

#6614 認証キー設定画面のモーダル内メッセージを多言語対応#6618
dotani1111 merged 1 commit intoEC-CUBE:4.3from
to19960425:fix/i18n-captcha-modal-message

Conversation

@to19960425
Copy link
Copy Markdown
Contributor

@to19960425 to19960425 commented Feb 6, 2026

概要(Overview・Refs Issue)

認証キー設定画面(authentication_setting.twig)のモーダル内に日本語がハードコードされていたメッセージを多言語対応。

Closes #6614

方針(Policy)

同テンプレート内の多言語対応済みメッセージと統一。
既存の admin.store.setting.get_api_key_info に準拠し、admin.store.setting.get_api_key_modal_info とした。

実装に関する補足(Appendix)

  • 変更対象は3ファイルのみ(テンプレート1、翻訳ファイル2)
  • 翻訳キーは既存キー get_api_key_info の直後に配置

テスト(Test)

  • 管理画面で [オーナーズストア] > [認証キー設定] を開き、「認証キーを取得する」ボタンをクリックしてモーダル内のメッセージが正しく表示されることを確認
  • 言語を英語に切り替え、英語メッセージが表示されることを確認

[ECCUBE_LOCALE: "ja"]
スクリーンショット 2026-02-06 094038

[ECCUBE_LOCALE: "en"]
スクリーンショット 2026-02-06 092225

相談(Discussion)

マイナーバージョン互換性保持のための制限事項チェックリスト

  • 既存機能の仕様変更はありません
  • フックポイントの呼び出しタイミングの変更はありません
  • フックポイントのパラメータの削除・データ型の変更はありません
  • twigファイルに渡しているパラメータの削除・データ型の変更はありません
  • Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
  • 入出力ファイル(CSVなど)のフォーマット変更はありません

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか
    • 権限を超えた操作が可能にならないか
    • 不要なファイルアップロードがないか
    • 外部へ公開されるファイルや機能の追加ではないか
    • テンプレートでのエスケープ漏れがないか

Summary by CodeRabbit

リリースノート

  • ドキュメンテーション
    • API キー認証設定画面のモーダルメッセージを多言語対応に更新しました。新規キー発行に関するガイダンステキストの英語および日本語翻訳を追加。既存キーを保有するユーザーに対し「発行せずに既存キーを設定してください」という指示が、各言語で正確に伝わるようになります。

テンプレートにハードコードされていた日本語メッセージを翻訳キーに置換し、
ja/en の翻訳ファイルに対応するエントリを追加。

Related to EC-CUBE#6614
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 6, 2026

📝 Walkthrough

Walkthrough

認証キー設定画面のモーダル内にあるハードコードされた日本語メッセージを多言語対応にしました。英語と日本語の翻訳キーを追加し、Twigテンプレートを修正して翻訳プレースホルダーを参照するように変更しました。

Changes

Cohort / File(s) Summary
翻訳キーの追加
src/Eccube/Resource/locale/messages.en.yaml, src/Eccube/Resource/locale/messages.ja.yaml
新しい翻訳キー admin.store.setting.get_api_key_modal_info を英語と日本語で各ファイルに追加しました。
テンプレートの更新
src/Eccube/Resource/template/admin/Store/authentication_setting.twig
モーダル内のハードコードされた日本語メッセージを翻訳キーを参照する形式に置き換えました。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 ✨

メッセージを翻訳の言葉に変えたよ
英語も日本語も仲良く並んで
モーダルの中で世界が広がった
認証キーよ、言語の壁を越えて
ユーザー皆が笑顔でアクセス!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR タイトルは認証キー設定画面のモーダルメッセージを多言語対応することという変更内容を的確に表現しており、GitHub issue #6614 と一致しています。
Linked Issues check ✅ Passed PR は issue #6614 の要件を完全に満たしており、硬くコードされた日本語メッセージを翻訳キーに置き換え、ja/en 翻訳ファイルに新しいキーを追加しました。
Out of Scope Changes check ✅ Passed 変更は issue #6614 の要件に完全に限定されており、不要な変更は含まれていません。3つのファイル(1つのテンプレート、2つの翻訳ファイル)のみが変更されました。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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

@dotani1111 dotani1111 added improvement 機能改善 refactor リファクタリング and removed improvement 機能改善 labels Feb 25, 2026
@dotani1111 dotani1111 added this to the 4.4.0 milestone Feb 25, 2026
@dotani1111
Copy link
Copy Markdown
Contributor

@to19960425
PRありがとうございます!
動作確認を進めます!

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.80%. Comparing base (37c5250) to head (7dd9e90).
⚠️ Report is 42 commits behind head on 4.3.

Additional details and impacted files
@@             Coverage Diff              @@
##                4.3    #6618      +/-   ##
============================================
- Coverage     78.84%   78.80%   -0.04%     
  Complexity     6631     6631              
============================================
  Files           475      475              
  Lines         26539    26539              
============================================
- Hits          20924    20915       -9     
- Misses         5615     5624       +9     
Flag Coverage Δ
Unit 78.80% <ø> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dotani1111
Copy link
Copy Markdown
Contributor

動作確認問題ありませんでした。

@dotani1111 dotani1111 merged commit b8716aa into EC-CUBE:4.3 Mar 5, 2026
100 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor リファクタリング Status: ready-for-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

認証キー設定画面の一部メッセージが多言語対応されていない。

3 participants