動画機能の表示制御をフィーチャーフラグに変更

[komagata]

Summary

• Rails.env.production? による動画機能の表示制御を movie_available? ヘルパーに変更
• development/test環境では常に有効、本番環境では Switchlet.enabled?(:movie) で制御
• smart_search_available? と同じパターン

変更箇所

• app/helpers/application_helper.rb: movie_available? ヘルパー追加
• app/views/pages/_doc_movie_header.html.slim: ヘッダータイトルの表示制御
• app/views/pages/_tabs.html.slim: 動画タブの表示制御
• app/helpers/page_tabs/practices_helper.rb: プラクティス内の動画タブの表示制御

Test plan

• development環境で「Docs・動画」ヘッダーと動画タブが表示されることを確認
• プラクティス詳細ページに動画タブが表示されることを確認

🤖 Generated with Claude Code

Summary by CodeRabbit

• 改善
  • 動画コンテンツの利用可能状況がより柔軟に制御できるようになりました。

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

ApplicationHelperにmovie_available?ヘルパーメソッドを追加し、ローカル環境またはSwitchletのmovieフィーチャーフラグが有効な場合に真を返すようにしました。既存の環境ベースの条件付きロジックをこのメソッドに置き換え、3つのテンプレートとヘルパーで統一的に使用するようにしました。

Changes

Cohort / File(s)	Summary
ヘルパーメソッド追加 app/helpers/application_helper.rb	movie_available?メソッドを新規追加。ローカル環境またはSwitchletのムービーフラグが有効な場合に真を返すロジック実装。
ビデオタブの条件付きロジック更新 app/helpers/page_tabs/practices_helper.rb, app/views/pages/_tabs.html.slim	Rails.env.production?の環境チェックからmovie_available?フェーチャーフラグチェックに置き換え。ビデオタブの表示制御をメソッドベースに統一。
ドキュメントヘッダーの条件付きロジック更新 app/views/pages/_doc_movie_header.html.slim	Rails.env.production?チェックをmovie_available?に置き換え。本番環境固有のコメントを削除し、フェーチャーフラグベースの制御に移行。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• 本番環境以外で動画一覧タブとプラクティスごとの動画タブを表示する #8905: ビデオタブの表示条件を環境チェックからフェーチャーフラグベースに切り替える変更で関連。

Suggested reviewers

• riq0h

Poem

🐰✨ 機能フラグで映像の門を開き、
環境チェックから優雅に別れを告げ、
ローカルでもリモートでも映像が光る、
条件分岐、すっきりと整い、
動画タブよ、輝きを放て！🎬

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	プルリクエストのタイトルは変更の主要な内容を明確に要約しており、動画機能の表示制御がフィーチャーフラグに変更されることを端的に伝えています。
Description check	✅ Passed	説明はテンプレートの必須セクション（Issue番号、概要、変更確認方法）が不完全ですが、Summary、変更箇所、Test planが詳しく記載されており、変更内容の理解には十分です。

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

✨ 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 feature/movie-feature-flag

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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