Skip to content

金額が前のものだったので修正#9261

Merged
machida merged 1 commit intomainfrom
price-check
Oct 17, 2025
Merged

金額が前のものだったので修正#9261
machida merged 1 commit intomainfrom
price-check

Conversation

@machida
Copy link
Copy Markdown
Member

@machida machida commented Oct 16, 2025

Summary by CodeRabbit

リリースノート

  • Chores
    • 企業研修プランの月額料金を更新しました。基本プランは月額32,780円、プレミアムプランは月額109,780円に改定されています(税込)。

@github-actions github-actions bot requested a review from komagata October 16, 2025 11:12
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Oct 16, 2025

Walkthrough

企業研修の料金表示を更新するマークアップのみの変更です。一般向け月額料金を29,800円から32,780円に、上位ティアの月額料金を99,800円から109,780円に変更しました(いずれも税込)。制御フローやエラーハンドリングへの影響はありません。

Changes

Cohort / File(s) Summary
企業研修料金更新
app/views/welcome/training.html.slim
料金表示を更新:一般月額 29,800円 → 32,780円、上位ティア月額 99,800円 → 109,780円(いずれも税込)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • fjordllc/bootcamp#9226: 同じビュー(app/views/welcome/training.html.slim)の月額・企業向け料金表示とマークアップを更新
  • fjordllc/bootcamp#9058: 同じファイルの企業研修料金表示を修正し、価格改定通知を追加
  • fjordllc/bootcamp#8775: 同じファイルのラベルおよび表記を変更

Suggested reviewers

  • okuramasafumi
  • komagata

Poem

🐰 料金もアップ、春の風
数字が踊る表の中
シンプルな変更、お手軽レビュー
ニンジンのように甘い更新♨️

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 unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch price-check

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: 0

🧹 Nitpick comments (1)
app/views/welcome/training.html.slim (1)

345-347: 税込表記と通貨記号の表記を統一することを検討してください。

ファイル内で税込の表記に揺れがあります:

  • Lines 323, 327: (税込)
  • Lines 345, 347: (税込み)

また、通貨記号の表記も統一されていません:

  • Lines 319-320, 327:
  • Lines 345, 347:

この不統一は本PRで導入されたものではなく、既存のものですが、将来的に統一することでユーザー体験の一貫性が向上します。

例えば、以下のように統一することを検討してください:

-                          | 月額¥32,780(税込み)
+                          | 月額32,780円(税込)
-                          | 月額¥109,780(税込み)
+                          | 月額109,780円(税込)
📜 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 883044e and be6a9ec.

📒 Files selected for processing (1)
  • app/views/welcome/training.html.slim (1 hunks)
⏰ 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)
app/views/welcome/training.html.slim (1)

345-347: 料金の更新が正しく反映されています!

一般利用と法人利用の料金が、ファイル内の他のセクション(Lines 318, 327)と一貫して更新されており、問題ありません。

@machida machida self-assigned this Oct 16, 2025
@machida machida merged commit ea11877 into main Oct 17, 2025
6 checks passed
@machida machida deleted the price-check branch October 17, 2025 00:14
@github-actions github-actions bot mentioned this pull request Oct 17, 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.

1 participant