日報一覧のプラクティスの絞り込む機能を修正

[matuaya]

Issue

• 日報一覧のプラクティスで絞り込む機能が動いていない #9217

概要

日報一覧の「プラクティスで絞り込む」でプラクティスを選択してもページがリロードせずに一覧が更新されないバグを修正しました。

変更確認方法

1. bug/fix-practice-filter-in-report-listをローカルに取り込む。
2. 任意のユーザーでログインする。
3. 左のメニュー一覧から「日報・ブログ」を選択し、日報一覧のページに行く。（http://localhost:3000/reports）
4. 「プラクティスで絞り込む」で「Terminalの基礎を覚える」を選択する。
5. 日報の一覧が更新され、関連の日報のみが表示されることを確認する。

Screenshot

UIに変化はないため省略します。

Summary by CodeRabbit

• Bug Fixes
  • レポート画面のフォームで、フィルタ操作（選択）の変更時にその値がGET送信されるようになりました。これにより選択項目が画面遷移や検索結果に正しく反映されます。

[coderabbitai]

Walkthrough

app/views/reports/index.html.slim の form_with 内で form_item ブロックのインデントを修正し、label_tag と select_tag がフォーム内に含まれるように移動しました。これにより該当の入力が GET 提出に含まれるようになります。

Changes

Cohort / File(s)	Summary
View: reports index form nesting app/views/reports/index.html.slim	form_item ブロックのインデントを修正し、label_tag と select_tag を form_with ブロック内にネスト。これにより該当フィールドがフォーム送信のスコープ内になる。

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant U as ユーザー
  participant B as ブラウザ (DOM)
  participant S as サーバー

  rect rgb(235, 245, 255)
  note over B: 旧フロー（フィールドがフォーム外）
  U->>B: 入力/選択
  U->>B: フォーム送信（GET）
  B->>S: GET（該当フィールドが欠落）
  S-->>B: 応答
  end

  rect rgb(230, 255, 230)
  note over B: 新フロー（フィールドがフォーム内）
  U->>B: 入力/選択
  U->>B: フォーム送信（GET）
  B->>S: GET（全フィールド含む）
  S-->>B: 応答
  end

Loading

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

• komagata
• sjabcdefin

Poem

ピョンと直してフォームに入る
風にのせた選択が帰る場所へ
クリックで舞い込む値たちよ
迷わず届け、サーバーの扉へ 🥕

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title Check	✅ Passed	タイトルは「日報一覧のプラクティスの絞り込む機能を修正」として、主な変更点であるプラクティス絞り込み機能のバグ修正を簡潔かつ明確に表現しており、要件を満たしています。
Description Check	✅ Passed	プルリクエスト説明はテンプレートの「Issue」「概要」「変更確認方法」「Screenshot」セクションを網羅しており、Issueリンクや検証手順が具体的に記載されています。UIに変化がない旨も明記されていて、実装内容と動作確認手順が明確です。
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 bug/fix-practice-filter-in-report-list

---

📜 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 17e5922 and 74af23d.

📒 Files selected for processing (1)

• app/views/reports/index.html.slim (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• app/views/reports/index.html.slim

⏰ 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

---

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.}

[coderabbitai]

Actionable comments posted: 1

📜 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 17e5922.

📒 Files selected for processing (1)

• app/views/reports/index.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

[matuaya]

@karlley
お疲れ様です😌
もし可能でしたらレビューをお願いできますでしょうか？🙇‍♀️
ご負担であれば遠慮なくおっしゃってください☺️

[karlley]

@matuaya
お疲れさまです！

もし可能でしたらレビューをお願いできますでしょうか？🙇‍♀️

他に1件レビュー中なので別の方に依頼していただいてもよろしいでしょうか？💦

[matuaya]

@karlley
そうでしたか💦
ご連絡ありがとうございます！🙏

[matuaya]

@smallmonkeykey
お疲れ様です☺️
もしご負担でなければレビューをお願いできますでしょうか？🙇‍♀️

[smallmonkeykey]

@matuaya
承知しました！3日以内にレビュー返します！

[matuaya]

@smallmonkeykey
ありがとうございます！よろしくお願いいたします🙏
全く急ぎではないので、いつでもご都合の良いタイミングでご確認いただけると嬉しいです☺️
一週間とかかかっても全然大丈夫です👍

[smallmonkeykey]

@matuaya
すみません、失念しており、予定より1時間遅れてしまいました🙇‍♂️
本当にすみませんでした。
動作確認ばっちりできました！
Approveいたします🙆

[matuaya]

@smallmonkeykey
ありがとうございました！☺️🙏
本当に全く急ぎではなかったので全然遅れたとか気にしないでください😊

[matuaya]

@okuramasafumi
チームメンバーのレビューが完了いたしましたのでご確認いただけると嬉しいです🙇‍♀️

[okuramasafumi]

lGTM!

なるほど、インデントのミスだったのですね、これはたしかにぱっと見で気づきづらい…

再発防止について少し考えてみましたが、form_withの中身が空（すぐ下の行が同じ深さのインデントになっている）なときにLinterで警告する、くらいしか思いつきませんでした…

[matuaya]

@okuramasafumi
レビューありがとうございました！
恥ずかしながら再発防止については考えていなかったので、参考になりました🙇‍♀️

@komagata
レビューが完了しましたのでマージお願いいたします🙇‍♀️

[komagata]

確認させていただきました。OKです〜🙆‍♂️