Skip to content

変更前:Railsエンジニア(Reスキル認定講座対応)コース → 変更後:給付金対象Railsエンジニアコース にコース名を変更した。#9240

Merged
komagata merged 2 commits intomainfrom
change-course-name
Oct 8, 2025

Conversation

@machida
Copy link
Copy Markdown
Member

@machida machida commented Oct 7, 2025

Summary by CodeRabbit

  • 新機能
    • 受講期間超過時の継続・支払い方法(24ヶ月以降の月額継続)に関する案内を追加
  • スタイル
    • コース名称を「給付金対象Railsエンジニアコース」に統一(申請フォーム、登録画面、コース選択、要件、料金、各詳細セクション、トップ/価格ページ等)
  • ドキュメント
    • コース詳細ページのタイトル、メタディスクリプション、画像代替テキストを最新名称に更新
  • テスト
    • サインアップ関連のシステムテストでフォーム選択子をより具体的に修正(form → form[name=user])

@github-actions github-actions bot requested a review from okuramasafumi October 7, 2025 06:05
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Oct 7, 2025

Walkthrough

各ビューの表示文言を「Railsエンジニア(Reスキル認定講座対応)コース」から「給付金対象Railsエンジニアコース」へ置換。認定講座ページのタイトル・meta・altや course_duration の文章追記(24ヶ月超過時の継続課金について)を含む。機能・ロジック変更なし。

Changes

Cohort / File(s) Summary
一般ウェルカム/価格/要件の文言更新
app/views/welcome/index.html.slim, app/views/welcome/choose_courses.html.slim, app/views/welcome/_course-select.html.slim, app/views/welcome/_pricing.html.slim, app/views/welcome/_requirements.html.slim, app/views/welcome/pricing.html.slim
表示テキスト内のコース名を「給付金対象Railsエンジニアコース」に統一。構造・ロジック変更なし。
認定講座(Rails)ページ群のタイトル・meta・alt更新
app/views/welcome/certified_reskill_courses/rails_developer_course/index.html.slim
ページタイトル、meta description、本文、画像altの文言を新名称に変更。
認定講座(Rails)パーシャルの文言更新
app/views/welcome/certified_reskill_courses/rails_developer_course/_course_details.html.slim, .../_course_fees.html.slim, .../_course_overview.html.slim, .../_course_requirements.html.slim
各パーシャル内のコース名表記を新名称へ置換。
認定講座(Rails)受講期間の文面調整
app/views/welcome/certified_reskill_courses/rails_developer_course/_course_duration.html.slim
コース名表記の更新、説明文の締めを調整し、24ヶ月超過時の継続受講と月額課金(Railsエンジニアコース相当)に関する追記を追加。
申請/登録フローの文言更新
app/views/grant_course_applications/new.html.slim, app/views/users/new.html.slim
申請フォーム/ユーザー登録内のコース名表記を新名称へ更新。
非ログインフッターのリンクラベル更新
app/views/shared/_not_logged_in_footer.html.slim
ナビゲーションのリンク表示名を新名称へ変更(リンク先は同一)。
テストのセレクタ調整
test/system/sign_up_test.rb
サインアップ系システムテストでフォーム選択子を form から form[name=user] に置換(アサーション精度向上)。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related issues

Possibly related PRs

  • 新料金表示に変更 #9226 — welcome と certified_reskill_courses のテンプレート群に対する変更が重複する可能性があるため関連性が高い。

Suggested reviewers

  • okuramasafumi
  • komagata

Poem

ぴょんと名を変えたビューの道
給付の風に耳を澄ませて
文だけそっと整えました
24ヶ月越えたら数える月よ
今日もビューに小さな春 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning プルリクエストの説明がまったく記載されておらずIssue番号、概要、変更確認方法、スクリーンショットの各必須セクションがすべて欠けているためテンプレートに準拠していません。 リポジトリで定義されたテンプレートに従いIssue番号と変更の概要を記載し動作確認手順と変更前後のスクリーンショットを追加してください。
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed タイトルはコース名変更というプルリクエストの主旨を明確に表しており変更前後の名称も具体的に示しているため主要な変更点を正確に要約しています。
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ 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 change-course-name

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c62d6f8 and 8911ad2.

📒 Files selected for processing (1)
  • test/system/sign_up_test.rb (1 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
**/*.rb

⚙️ CodeRabbit configuration file

**/*.rb: # refactoring

  • まずFat Controllerを避け、次にFat Modelを避ける。
  • Serviceクラスの乱用を避ける。
  • controller concernを作ろうとしたらPORO(Plain Old Ruby Object)やActiveRecordモデルでの実装で代替できないか検討する。

Rails Patterns

  • ViewHelperにメソッドを実装する時にはまずDecoratorパターンを使うことを検討する。(active_decorator gemを導入しているのでそれを使う)
  • 複雑なActiveRecordクエリがあり、再利用できそうな場合はQueryObjectパターンを検討する。(rails-patterns gemを導入しているのでそれのQuery機能を使う)
  • Viewにpartialを作る場合はViewComponentを使うことを検討する。
  • 複数のActiveRecordモデルを操作する1つの責務がある時や外部APIとやりとりする処理がある場合にはInteractorオブジェクトパターンを検討する。(interactor gemを導入しているのでそれを使う)
  • 複数のInteractorを実行するような処理がある場合Organizerオブジェクトパターンを検討する。(interactor gemを導入しており、その中にOrganizerの機能があるので使う)

Files:

  • test/system/sign_up_test.rb
test/**/*

⚙️ CodeRabbit configuration file

test/**/*: # Test

  • TestCase名は英語で書く。
  • どうしても避けられない時以外にsystem testでsleepは使わない。

Unit Test

model, helper, decorator, view_componentについてはメソッドを追加した場合は必ず対応したUnit TestのTestCaseを1つは書く。

Files:

  • test/system/sign_up_test.rb
🧬 Code graph analysis (1)
test/system/sign_up_test.rb (2)
test/system/products_test.rb (1)
  • test (5-722)
test/system/practices_test.rb (1)
  • test (5-329)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: build_and_test
🔇 Additional comments (1)
test/system/sign_up_test.rb (1)

295-295: LGTM! より具体的なセレクタへの改善

'form' から 'form[name=user]' への変更により、テストの精度が向上しています。ページに複数のフォームが存在する場合でも、ユーザー登録フォームを確実に選択できるようになり、ファイル内の他のテスト(Lines 18, 45, 71など)や関連テストファイル(products_test.rb)との一貫性も保たれています。

Also applies to: 301-301, 307-307, 313-313, 319-319

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 RuboCop (1.81.1)
test/system/sign_up_test.rb

rubocop-minitest extension supports plugin, specify plugins: rubocop-minitest instead of require: rubocop-minitest in /.rubocop.yml.
For more information, see https://docs.rubocop.org/rubocop/plugin_migration_guide.html.
rubocop-capybara extension supports plugin, specify plugins: rubocop-capybara instead of require: rubocop-capybara in /.rubocop.yml.
For more information, see https://docs.rubocop.org/rubocop/plugin_migration_guide.html.
Unable to find gem rubocop-fjord; is the gem installed? Gem::MissingSpecError
/var/lib/gems/3.1.0/gems/rubocop-1.81.1/lib/rubocop/config_loader_resolver.rb:309:in rescue in gem_config_path' /var/lib/gems/3.1.0/gems/rubocop-1.81.1/lib/rubocop/config_loader_resolver.rb:293:in gem_config_path'
/var/lib/gems/3.1.0/gems/rubocop-1.81.1/lib/rubocop/config_loader_resolver.rb:84:in block (2 levels) in resolve_inheritance_from_gems' /var/lib/gems/3.1.0/gems/rubocop-1.81.1/lib/rubocop/config_loader_resolver.rb:82:in reverse_each'
/var/lib/gems/3.1.0/g

... [truncated 255 characters] ...

fig_loader_resolver.rb:76:in resolve_inheritance_from_gems' /var/lib/gems/3.1.0/gems/rubocop-1.81.1/lib/rubocop/config_loader.rb:58:in load_file'
/var/lib/gems/3.1.0/gems/rubocop-1.81.1/lib/rubocop/config_store.rb:34:in options_config=' /var/lib/gems/3.1.0/gems/rubocop-1.81.1/lib/rubocop/config_store.rb:29:in apply_options!'
/var/lib/gems/3.1.0/gems/rubocop-1.81.1/lib/rubocop/cli.rb:167:in act_on_options' /var/lib/gems/3.1.0/gems/rubocop-1.81.1/lib/rubocop/cli.rb:49:in block in run'
/var/lib/gems/3.1.0/gems/rubocop-1.81.1/lib/rubocop/cli.rb:87:in profile_if_needed' /var/lib/gems/3.1.0/gems/rubocop-1.81.1/lib/rubocop/cli.rb:45:in run'
/var/lib/gems/3.1.0/gems/rubocop-1.81.1/exe/rubocop:15:in <top (required)>' /usr/local/bin/rubocop:25:in load'
/usr/local/bin/rubocop:25:in `

'


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1a61fa1 and c62d6f8.

📒 Files selected for processing (15)
  • app/views/grant_course_applications/new.html.slim (1 hunks)
  • app/views/shared/_not_logged_in_footer.html.slim (1 hunks)
  • app/views/users/new.html.slim (1 hunks)
  • app/views/welcome/_course-select.html.slim (2 hunks)
  • app/views/welcome/_pricing.html.slim (1 hunks)
  • app/views/welcome/_requirements.html.slim (1 hunks)
  • app/views/welcome/certified_reskill_courses/rails_developer_course/_course_details.html.slim (1 hunks)
  • app/views/welcome/certified_reskill_courses/rails_developer_course/_course_duration.html.slim (1 hunks)
  • app/views/welcome/certified_reskill_courses/rails_developer_course/_course_fees.html.slim (1 hunks)
  • app/views/welcome/certified_reskill_courses/rails_developer_course/_course_overview.html.slim (1 hunks)
  • app/views/welcome/certified_reskill_courses/rails_developer_course/_course_requirements.html.slim (1 hunks)
  • app/views/welcome/certified_reskill_courses/rails_developer_course/index.html.slim (3 hunks)
  • app/views/welcome/choose_courses.html.slim (1 hunks)
  • app/views/welcome/index.html.slim (1 hunks)
  • app/views/welcome/pricing.html.slim (1 hunks)
🔇 Additional comments (15)
app/views/welcome/choose_courses.html.slim (1)

42-42: コース名の更新を確認しました。

テキストの更新が正しく反映されています。PRの目的と一致しています。

app/views/welcome/certified_reskill_courses/rails_developer_course/_course_details.html.slim (1)

15-15: コース名の更新を確認しました。

テキストの更新が正しく反映されています。

app/views/welcome/certified_reskill_courses/rails_developer_course/_course_fees.html.slim (1)

33-33: コース名の更新を確認しました。

テキストの更新が正しく反映されています。

app/views/welcome/certified_reskill_courses/rails_developer_course/_course_overview.html.slim (1)

19-19: コース名の更新を確認しました。

テキストの更新が正しく反映されています。

app/views/shared/_not_logged_in_footer.html.slim (1)

37-37: コース名の更新を確認しました。

テキストの更新が正しく反映されています。

app/views/welcome/_requirements.html.slim (1)

33-33: コース名の更新を確認しました。

テキストの更新が正しく反映されています。

app/views/welcome/index.html.slim (1)

26-26: コース名の更新を確認しました。

テキストの更新が正しく反映されています。

app/views/welcome/_pricing.html.slim (1)

45-45: LGTM!

コース名の変更が正しく反映されています。

app/views/grant_course_applications/new.html.slim (1)

12-12: LGTM!

コース名が『』で強調され、わかりやすく更新されています。

app/views/welcome/pricing.html.slim (1)

71-71: LGTM!

コース名の変更が正しく反映されています。

app/views/welcome/certified_reskill_courses/rails_developer_course/_course_requirements.html.slim (1)

16-16: LGTM!

コース名の変更が正しく反映されています。

app/views/welcome/certified_reskill_courses/rails_developer_course/index.html.slim (1)

1-1: LGTM!

タイトル、メタディスクリプション、本文のコース名変更が正しく反映されています。

Also applies to: 3-3, 25-25

app/views/welcome/_course-select.html.slim (1)

20-20: LGTM!

コース選択画面での各箇所のコース名変更が正しく反映されています。説明文、カードタイトル、詳細説明のすべてで一貫性が保たれています。

Also applies to: 66-66, 71-71

app/views/welcome/certified_reskill_courses/rails_developer_course/_course_duration.html.slim (2)

30-30: LGTM!

コース名の変更が正しく反映されています。


34-40: LGTM! 受講期間超過時の対応が明確になりました。

24ヶ月で卒業に至らなかった場合の延長方法と課金方式が追加され、受講者にとって重要な情報が明確に記載されています。Railsエンジニアコースへのリンクも適切に設置されており、詳細を確認できる導線が整っています。

@komagata komagata merged commit 6211370 into main Oct 8, 2025
3 checks passed
@komagata komagata deleted the change-course-name branch October 8, 2025 04:34
@github-actions github-actions bot mentioned this pull request Oct 8, 2025
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants