Merged
Conversation
Contributor
Author
|
遅くなりましたが、文言修正のPR作成しました。 参考: #8754 (comment) |
machida
force-pushed
the
feature/change-the-wording-corporate-use
branch
from
ebb2473 to
be7e3dd
Compare
Member
|
@tyrrell-IH おまたせしてすいません！全体の文言を調整しました。これでレビューへお願いします。 |
## Walkthrough
「法人利用」という表現を「企業研修代行」に統一するため、コントローラ、ビューファイル、FAQデータ、テストなどの文言やカテゴリ名が変更されました。FAQ取得処理はカテゴリ未存在時にも安全に動作するよう修正されています。
## Changes
| ファイル・グループ | 変更概要 |
|------------------------------------------|--------------------------------------------------------------------------------------------|
| app/controllers/welcome_controller.rb | FAQカテゴリ名を「法人利用について」から「企業研修代行について」へ変更、FAQ取得処理を安全化 |
| app/views/layouts/lp/_lp_header.html.slim<br>app/views/welcome/_conditions.html.slim<br>app/views/welcome/pricing.html.slim<br>app/views/welcome/training.html.slim | ナビゲーションや各種説明文の「法人利用」を「企業研修代行」など新表現に統一 |
| db/fixtures/faq_categories.yml<br>test/fixtures/faq_categories.yml | FAQカテゴリ名を「企業研修代行について」に変更 |
| db/fixtures/faqs.yml | 法人利用・料金に関するFAQの説明文・質問文を「企業研修代行」等に修正 |
| test/system/require_login_test.rb | '/training'ページのテスト期待値を新しいタイトル・説明文に変更 |
| test/system/welcome_test.rb | '/training'ページのタイトル・メタタグのテスト期待値を新表現に変更 |
## Sequence Diagram(s)
```mermaid
sequenceDiagram
participant User
participant WelcomeController
participant FAQCategory
participant View
User->>WelcomeController: /training にアクセス
WelcomeController->>FAQCategory: find_by(name: '企業研修代行について')
FAQCategory-->>WelcomeController: FAQCategoryインスタンス or nil
WelcomeController->>WelcomeController: @faqs = FAQCategory&.faqs || []
WelcomeController->>View: @faqs, 他データを渡す
View-->>User: 企業研修代行ページを表示Possibly related issues
Suggested reviewers
Poem
|
tyrrell-IH
changed the title
Contributor
Author
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
app/views/welcome/_conditions.html.slim (1)
14-15: 日本語の主語・対象を統一するとさらに読みやすくなります
「（法人でのご利用のみ、請求書発行に対応しています。）」 → 「※法人でのご利用のみ請求書発行に対応しています。」のように、読点を減らして注釈マークを付けると LP 上で視認性が上がります。
任意ですがご検討ください。
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (10)
app/controllers/welcome_controller.rb(2 hunks)app/views/layouts/lp/_lp_header.html.slim(1 hunks)app/views/welcome/_conditions.html.slim(1 hunks)app/views/welcome/pricing.html.slim(1 hunks)app/views/welcome/training.html.slim(6 hunks)db/fixtures/faq_categories.yml(1 hunks)db/fixtures/faqs.yml(1 hunks)test/fixtures/faq_categories.yml(1 hunks)test/system/require_login_test.rb(1 hunks)test/system/welcome_test.rb(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
app/views/welcome/training.html.slim (1)
Learnt from: tyrrell-IH
PR: fjordllc/bootcamp#8807
File: app/views/welcome/job_support.html.slim:391-392
Timestamp: 2025-06-21T22:30:20.116Z
Learning: このプロジェクトにはRailsの組み込みmarkdownヘルパが利用できないため、markdownのHTML変換には自前でヘルパを実装する必要がある。
🔇 Additional comments (14)
db/fixtures/faq_categories.yml (1)
25-27: 名称変更に伴う参照漏れを確認してください
faq_categories7のnameを「企業研修代行について」に変更したことで、コード側で 文字列比較 している箇所が残っていると FAQ が表示されなくなります。
定数・スコープ・テスト等に"法人利用について"がハードコードされていないか一度 grep で確認することを推奨します。test/fixtures/faq_categories.yml (1)
37-40: テストフィクスチャも更新済み – 👍
アプリ側と同じ名称に揃えられているため、フィクスチャ読み込み時の不整合は発生しません。db/fixtures/faqs.yml (1)
185-190: 長文回答内の改行エスケープに注意
インライン改行を含む日本語文を|-ブロックで書く場合、末尾のスペース欠落やインデント誤りで YAML パースエラーが発生しやすいです。
ローカルでrails db:fixtures:loadを実行して問題なく読み込めるか必ず確認してください。app/views/layouts/lp/_lp_header.html.slim (1)
32-35: ナビゲーションラベル変更 OK
リンク先 (training_path) はそのままで、表示文言のみ更新されていることを確認しました。ビューファイル間の整合も取れているため問題ありません。app/views/welcome/pricing.html.slim (1)
58-58: 文言の改善が適切です「法人でのご利用の場合は」への変更により、より丁寧で適切な表現になっています。企業向けサービスの説明として相応しい敬語表現です。
test/system/welcome_test.rb (1)
29-31: テストが新しい文言と正しく同期されていますページタイトルとメタタグの期待値が「企業研修代行」に適切に更新されており、ビューの変更と一致しています。
test/system/require_login_test.rb (1)
73-73: テスト内容が更新されたページコンテンツと一致しています期待値の文言が「FBCの法人向けプログラミング研修」に更新され、研修ページの新しいコンテンツと正しく対応しています。
app/views/welcome/training.html.slim (5)
2-2: ページタイトルが新しい文言体系と一致しています「企業研修代行」への変更により、サービス内容がより明確に表現されています。
33-33: 画像のalt属性も適切に更新されていますアクセシビリティを考慮して、alt属性の文言も新しい用語に合わせて更新されています。
279-279: 社内メンター説明文の文言が改善されています「企業研修代行他、法人でのご利用の場合は」という表現により、サービス対象がより具体的かつ丁寧に説明されています。
327-327: 料金セクションの表記が統一されています見出しとテーブルヘッダーの文言が「法人でのご利用料金」「法人利用」に統一され、より分かりやすい表現になっています。
Also applies to: 359-359
737-737: FAQ セクションのタイトルが更新されています「企業研修代行について」への変更により、FAQ カテゴリーとの整合性が保たれています。
app/controllers/welcome_controller.rb (2)
8-8: FAQ カテゴリー定数が新しい文言体系と一致しています定数名の更新により、全体的な用語統一が完了しています。
36-36: FAQの取得処理が安全になりましたセーフナビゲーション演算子（
&.）とフォールバック処理（|| []）の追加により、FAQカテゴリーが存在しない場合でもエラーが発生せず、空の配列が返されるようになっています。これは堅牢性の向上として優れた改善です。
Comment on lines
+196
to
202
| question: 法人での利用料が高いのはなぜですか？ | ||
| answer: |- | ||
| 法人利用の料金が高い理由は2つあります。 | ||
|
|
||
| まず、法人利用では業務時間内にFBCでの学習を行うため、一般利用者よりも多くのアウトプットが求められます。これにより、メンターのリソースがより多く必要とされるためです。 | ||
| まず、企業研修代行では業務時間内にFBCでの学習を行うため、一般利用者よりも多くのアウトプットが求められます。これにより、メンターのリソースがより多く必要とされるためです。 | ||
|
|
||
| 二つ目の理由は、FBCが月額の利用料と提携企業への卒業生紹介の際に発生する紹介料によって成り立っている点です。法人利用の場合、後者の収益が立たないため、一般利用よりも高額が設定されています。 |
There was a problem hiding this comment.
🛠️ Refactor suggestion
表記ゆれが残っています
質問文は「法人での利用料が高いのはなぜですか？」に修正されていますが、回答冒頭が依然として「法人利用の料金…」となっています。
用語を「企業研修代行」または「法人での利用」に統一してください。
- 法人利用の料金が高い理由は2つあります。
+ 企業研修代行（法人利用）の料金が高い理由は2つあります。同様に 202 行目の「法人利用の場合」も揃えると一貫性が保てます。
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| question: 法人での利用料が高いのはなぜですか？ | |
| answer: |- | |
| 法人利用の料金が高い理由は2つあります。 | |
| まず、法人利用では業務時間内にFBCでの学習を行うため、一般利用者よりも多くのアウトプットが求められます。これにより、メンターのリソースがより多く必要とされるためです。 | |
| まず、企業研修代行では業務時間内にFBCでの学習を行うため、一般利用者よりも多くのアウトプットが求められます。これにより、メンターのリソースがより多く必要とされるためです。 | |
| 二つ目の理由は、FBCが月額の利用料と提携企業への卒業生紹介の際に発生する紹介料によって成り立っている点です。法人利用の場合、後者の収益が立たないため、一般利用よりも高額が設定されています。 | |
| question: 法人での利用料が高いのはなぜですか？ | |
| answer: |- | |
| 企業研修代行（法人利用）の料金が高い理由は2つあります。 | |
| まず、企業研修代行では業務時間内にFBCでの学習を行うため、一般利用者よりも多くのアウトプットが求められます。これにより、メンターのリソースがより多く必要とされるためです。 | |
| 二つ目の理由は、FBCが月額の利用料と提携企業への卒業生紹介の際に発生する紹介料によって成り立っている点です。法人利用の場合、後者の収益が立たないため、一般利用よりも高額が設定されています。 |
🤖 Prompt for AI Agents
In db/fixtures/faqs.yml around lines 196 to 202, the terminology in the answer
is inconsistent with the question. Change all instances of "法人利用" in the answer
to match the question's term "法人での利用" or use "企業研修代行" consistently throughout.
Ensure that the phrase on line 202 "法人利用の場合" is also updated to maintain uniform
terminology for clarity and consistency.
Contributor
Author
There was a problem hiding this comment.
サービス運営者の確認をとっていますので、今回はこの表記のままリリースします。
There was a problem hiding this comment.
@tyrrell-IH 承知いたしました。サービス運営者の確認が取れているということであれば、現在の表記で問題ありません。リリース作業を進めてください。
|
Note Generated docstrings for this pull request at #8866 |
coderabbitai bot
added a commit
that referenced
this pull request
Jun 28, 2025
Docstrings generation was requested by @tyrrell-IH. * #8775 (comment) The following files were modified: * `app/controllers/welcome_controller.rb`
Contributor
Author
|
@riq0h |
riq0h
approved these changes
Jun 29, 2025
Contributor
riq0h
left a comment
riq0h
left a comment
There was a problem hiding this comment.
確認しました！ OKと思われるのでApproveさせていただきます🙇
Contributor
Author
komagata
reviewed
Jul 2, 2025
|
|
||
| def training | ||
| @faqs = FAQCategory.find_by(name: FAQ_CATEGORY_NAME).faqs | ||
| @faqs = FAQCategory.find_by(name: FAQ_CATEGORY_NAME)&.faqs || [] |
Contributor
Author
There was a problem hiding this comment.
町田さん作成のcommit: fdb1259で修正された箇所なので推測にはなりますが、
FAQ_CATEGORY_NAMEにFAQカテゴリー登録されていない名称を入れてしまうとFAQCategory.find_by(name: FAQ_CATEGORY_NAME)がnilを返し、nil.faqsでエラーが発生してしまう
のでそれを避けるための処理だと思います。
このPRでその修正を加えるべきではないということであれば当初の状態に戻します🙏
この修正の意図が問題ないのなら、
def training
category = FAQCategory.find_by(name: FAQ_CATEGORY_NAME)
@faqs = category&.faqs || FAQ.none
endに変更したいのですが、いかがでしょうか？
一旦commitはしておきました。
commit: 8ee55a1
空の配列を返すよりFAQ.noneでFAQ::ActiveRecord_Relationを返す方が、戻り値が統一できて良いのではと思いました。
Member
There was a problem hiding this comment.
tyrrell-IH
force-pushed
the
feature/change-the-wording-corporate-use
branch
3 times, most recently
from
72a8d6f to
139866b
Compare
tyrrell-IH
force-pushed
the
feature/change-the-wording-corporate-use
branch
from
139866b to
343c7b5
Compare
tyrrell-IH
force-pushed
the
feature/change-the-wording-corporate-use
branch
from
343c7b5 to
8ee55a1
Compare
87 tasks
Merged
This was referenced Aug 16, 2025
Merged
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue
概要
bootcampアプリ内で使用されている「法人利用」という文言を「企業研修代行」に修正しました。
修正した後の文章は、町田さんによるチェックの後、文章を調整するための修正が加えられています。
経緯
変更確認方法
1.
feature/change-the-wording-corporate-useをローカルに取り込み、当該ブランチに切り替える。2. セットアップ
3. fixtureの読み込み（当該PRにはfixturesファイルの修正があります）
4. ローカルでサーバーを起動する
5. 修正された項目の確認
以下に修正した箇所のURLと修正前後を比較したスクショを貼っています。
そちらを参考に開発用bootcampアプリ内で適切に文言の修正がなされているか確認してください。
1. /training
http://localhost:3000/training
スクリーンショット
header部分
修正前
修正後
メニューバー
修正前
修正後
imgタグalt属性
修正前
修正後
文章1
修正前
修正後
文章2
修正前
修正後
文章3
修正前
修正後
文章4
修正前
修正後
2. /(welcome)
http://localhost:3000/ 以下の変更箇所
スクリーンショット
修正前
修正後
3. /pricing
http://localhost:3000/pricing 以下の変更箇所
スクリーンショット
修正前
修正後
4. fixturesファイル
faq_categoriesテーブルのname属性に「法人利用」という文言が使われていたのでその点を修正。内部的な修正なのでスクリーンショットなし。
commit: 9d32968
5. テスト
/training以下に研修利用という文言がなくなったために通らなくなったテストを修正しました。内部的な修正のためスクリーンショットなし。
commit: 229dd90
Summary by CodeRabbit
新機能
バグ修正
ドキュメント
リファクタ
スタイル
テスト
その他