Conversation
Walkthrough表示価格を月額29,800円から月額32,780円（税込）へ更新。法人向けを99,800円から109,780円（税込）へ変更。複数ビューの価格表示・メタ説明の文言修正、価格表示用LPスタイルの拡張、一部価格改定告知ブロックの削除、関連テスト期待値の更新を実施。 Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested reviewers
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches
🧪 Generate unit tests
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro 📒 Files selected for processing (12)
🚧 Files skipped from review as they are similar to previous changes (9)
🧰 Additional context used📓 Path-based instructions (2)**/*.rb⚙️ CodeRabbit configuration file
Files:
test/**/*⚙️ CodeRabbit configuration file
Files:
🧠 Learnings (1)📚 Learning: 2025-09-12T01:00:58.452ZApplied to files:
⏰ 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)
🔇 Additional comments (4)
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)app/helpers/meta_tags_helper.rbrubocop-minitest extension supports plugin, specify ... [truncated 255 characters] ... fig_loader_resolver.rb:76:in test/system/articles_test.rbrubocop-minitest extension supports plugin, specify ... [truncated 255 characters] ... fig_loader_resolver.rb:76:in test/system/admin/campaigns_test.rbrubocop-minitest extension supports plugin, specify ... [truncated 255 characters] ... fig_loader_resolver.rb:76: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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
app/views/welcome/training.html.slim (1)
345-347: 価格表の更新漏れ「一般利用と法人利用の違い」比較表の価格が旧料金のままです:
- 一般利用: 月額￥29,800（税込み）→ 月額￥32,780（税込）に更新が必要
- 法人利用: 月額￥99,800（税込み）→ 月額￥109,780（税込）に更新が必要
また、表記を他の箇所と統一するため「（税込み）」を「（税込）」に変更することを推奨します。
以下のdiffを適用して価格を更新してください:
td.text-right | 利用料 td.text-center - | 月額￥29,800(税込み) + | 月額￥32,780(税込) td.text-center - | 月額￥99,800(税込み) + | 月額￥109,780(税込)
🧹 Nitpick comments (1)
app/javascript/stylesheets/lp/blocks/lp/_lp-price.sass (1)
64-78:.lp-price__per-personが未使用です
.lp-price__taxと.lp-price__total-valueはビューで使用されていますが、.lp-price__per-personへの参照が見つかりませんでした。不要であれば削除、または使用意図をコメントで追記してください。
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (12)
app/helpers/meta_tags_helper.rb(2 hunks)app/javascript/stylesheets/lp/blocks/lp/_lp-price.sass(3 hunks)app/javascript/stylesheets/lp/layouts/_l-cards.sass(1 hunks)app/views/welcome/_pricing.html.slim(1 hunks)app/views/welcome/certified_reskill_courses/rails_developer_course/_course_fees.html.slim(4 hunks)app/views/welcome/certified_reskill_courses/rails_developer_course/_course_overview.html.slim(2 hunks)app/views/welcome/index.html.slim(1 hunks)app/views/welcome/law.html.slim(1 hunks)app/views/welcome/pricing.html.slim(2 hunks)app/views/welcome/training.html.slim(2 hunks)test/system/admin/campaigns_test.rb(1 hunks)test/system/articles_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モデルを操作する１つの責務がある時や外部APIとやりとりする処理がある場合にはInteractorオブジェクトパターンを検討する。（interactor gemを導入しているのでそれを使う）
- 複数のInteractorを実行するような処理がある場合Organizerオブジェクトパターンを検討する。（interactor gemを導入しており、その中にOrganizerの機能があるので使う）
Files:
test/system/admin/campaigns_test.rbapp/helpers/meta_tags_helper.rbtest/system/articles_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/admin/campaigns_test.rbtest/system/articles_test.rb
🧠 Learnings (1)
📚 Learning: 2025-09-12T01:00:58.452Z
Learnt from: Miya096jp
PR: fjordllc/bootcamp#9102
File: app/views/mentor/buzzes/edit.html.slim:1-0
Timestamp: 2025-09-12T01:00:58.452Z
Learning: app/views/mentor/buzzes/edit.html.slim では `- title` と `- set_meta_tags` の両方が正しく設定されており、タイトルは削除されていない。変更内容はメタディスクリプションの追加のみ。
Applied to files:
test/system/articles_test.rb
⏰ 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 (26)
app/views/welcome/law.html.slim (1)
3-3: LGTM!メタディスクリプションの価格表示が新料金（32,780円（税込））に正しく更新されています。
app/views/welcome/certified_reskill_courses/rails_developer_course/_course_overview.html.slim (2)
76-78: LGTM!受講料が新価格（786,720円（税込））に更新され、表記も「授業料」から「受講料」に統一されています。一貫性が向上しています。
87-88: LGTM!必須教材費について、セールや書籍形式による金額変動の可能性を説明する文が追加されています。ユーザーへの情報提供として適切です。
app/views/welcome/index.html.slim (2)
2-2: LGTM!メタディスクリプションの価格が新料金（32,780円（税込））に正しく更新されています。
4-5: 問題なし: レイアウトにyield :head_lastが存在しないため、head_lastブロック削除の影響はありませんapp/views/welcome/_pricing.html.slim (2)
17-22: LGTM!月額料金が新価格（32,780円）に更新され、税込表記が適切に構造化されています。
26-26: LGTM!法人利用価格も新料金（109,780円（税込））に正しく更新されています。
app/views/welcome/certified_reskill_courses/rails_developer_course/_course_fees.html.slim (3)
23-29: LGTM!料金表示が「受講料」に一本化され、新価格（786,720円（税込））が正しく表示されています。シンプルで分かりやすい構成です。
51-58: LGTM!給付金の計算が正確です（786,720円の80% = 629,376円）。説明文も税込価格を参照するように更新されています。
69-76: LGTM!自己負担額の計算が正確です（786,720円 - 629,376円 = 157,344円）。
app/views/welcome/training.html.slim (3)
303-303: LGTM!コンテナサイズが
is-mdからis-lgに変更され、レイアウトが改善されています。
312-323: LGTM!法人利用料金の表示構造が改善され、「1名につき月額」の表記が追加されています。新価格（109,780円（税込））も正しく表示されています。
325-327: LGTM!一般利用価格への参照が新料金（32,780円（税込））に更新されています。
app/javascript/stylesheets/lp/layouts/_l-cards.sass (1)
38-38: LGTM!カードアイテムの最大幅を
calc(100vw - 2rem)に変更することで、左右に1remのマージンが確保され、レスポンシブレイアウトが改善されています。app/helpers/meta_tags_helper.rb (2)
10-10: LGTM!デフォルトメタタグの説明文が新料金（32,780円（税込））に正しく更新されています。
34-34: LGTM!welcomeページ用メタタグの説明文も新料金（32,780円（税込））に統一されています。
test/system/admin/campaigns_test.rb (1)
88-88: 価格表示の更新が正しく反映されています。コメントアウトされたテスト内のアサーションが新しい価格「月額32,780円（税込）」に更新されており、将来このテストが有効化された際の整合性が保たれています。
test/system/articles_test.rb (1)
268-268: メタディスクリプションの期待値が新料金に更新されています。テストが新しい価格「月額32,780円（税込）」を期待するよう正しく更新されています。これにより、
app/helpers/meta_tags_helper.rbでの変更と整合性が保たれています。app/views/welcome/pricing.html.slim (3)
6-6: メタディスクリプションが新料金に更新されています。説明文が「月額32,780円（税込）」に正しく更新されており、
app/helpers/meta_tags_helper.rbの変更と一貫性があります。
48-48: 表示価格が新料金に更新されています。月額料金が32,780円に正しく更新されています。税込表示はLine 53で別途表示されており、適切に分離されています。
57-57: 法人向け料金が新料金に更新されています。法人利用の月額料金が109,780円（税込）に正しく更新されています。個人向け料金との整合性が保たれています。
app/javascript/stylesheets/lp/blocks/lp/_lp-price.sass (5)
12-13: 新しい価格表示バリエーションが追加されています。
.is-totalモディファイアが追加され、合計金額表示用のより狭いギャップ（.25em）が設定されています。これにより、価格表示のバリエーションが拡張されています。
19-27: 税込表示のフォントサイズが拡大されています。
.lp-price__taxのフォントサイズが各ブレークポイントで拡大され、税込表示の視認性が向上しています。新料金表示における税込情報の重要性を反映した変更です。
45-45: 価格コンポーネントの折り返しに対応しています。
flex-wrap: wrapの追加により、小さい画面で価格要素が適切に折り返されるようになり、レスポンシブ対応が改善されています。
57-57: 価格詳細のフォントサイズが拡大されています。
.lp-price__detailsのフォントサイズが1.25emから2.25emに拡大され、税込などの詳細情報がより読みやすくなっています。
81-81: 価格注記の行間が調整されています。フォントサイズ指定からline-heightへの変更により、小さい画面でも適切な行間が保たれ、可読性が向上しています。
1 participant
Summary by CodeRabbit