Skip to content

近日開催のペアワークがメンター、アドバイザーのダッシュボードでも表示されるようにした#9755

Open
mousu-a wants to merge 2 commits intomainfrom
bug/fix-upcoming-pair-works
Open

近日開催のペアワークがメンター、アドバイザーのダッシュボードでも表示されるようにした#9755
mousu-a wants to merge 2 commits intomainfrom
bug/fix-upcoming-pair-works

Conversation

@mousu-a
Copy link
Copy Markdown
Contributor

@mousu-a mousu-a commented Mar 7, 2026

Issue

概要

ペアワーク機能にて、近日開催のペアワークが受講生のダッシュボードにしか表示されていなかったのでメンター、アドバイザーのダッシュボードでも表示されるようにしたPRです。

変更確認方法

  1. bug/fix-upcoming-pair-worksをローカルに取り込む
  2. kimuraでログイン
  3. 新規作成にて各項目を入力し、ペアワークを作成
    1. スケジュールには、明日〜明後日辺りのどこかにチェックを入れてください(近日開催のペアワークの動作確認をするためです)
  4. komagataでログイン
  5. 先ほど作成したペアワークのページに行き、スケジュール表の✅をクリックし、ペア確定する
  6. ダッシュボードにアクセスし、近日開催のペアワークが表示されているのを確認
    7. bin/rails db:seedを実行する
  7. advijirouでログイン
  8. ダッシュボードにて、下にスクロールし、近日開催のペアワークが表示されているのを確認

Screenshot

デザイン確認後に追加します!

変更前

変更後

@mousu-a mousu-a self-assigned this Mar 7, 2026
@github-actions github-actions bot requested a review from komagata March 7, 2026 00:37
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 7, 2026

📝 Walkthrough

Walkthrough

ダッシュボードビュー(アドバイザーおよびメンター)に条件付きでupcoming_pair_worksパーシャルをレンダーするロジックを追加し、フィクスチャファイル内のペアワークレコードの参照を更新しました。

Changes

Cohort / File(s) Summary
ダッシュボードビューの更新
app/views/home/_adviser_dashboard.html.slim, app/views/home/_mentor_dashboard.html.slim
@upcoming_pair_worksが存在する場合、upcoming_pair_worksパーシャルを条件付きでレンダーするコードを各ダッシュボードに追加しました。
フィクスチャデータの修正
db/fixtures/pair_works.yml
pair_work3レコードのbuddyフィールド値をsotugyouからadvijirouに変更しました。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 ウサギの足跡をたどれば、
ダッシュボードに新しい景色、
ペアワークのパーシャルが舞い降りて、
条件付きで光を灯す、
シンプルな変更、大きな笑顔!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed プルリクエストのタイトルは、メンターとアドバイザーのダッシュボードに近日開催のペアワークを表示するという主要な変更を明確に要約しており、変更内容と完全に関連しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed Pull requestの説明は必須セクションをほぼ完備しており、Issue番号、概要、変更確認方法が記載されています。

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bug/fix-upcoming-pair-works

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.

🧹 Nitpick comments (1)
app/views/home/_mentor_dashboard.html.slim (1)

28-29: この表示分岐は回帰テストで固定しておきたいです。

今回の不具合は role 別ダッシュボードでの描画漏れなので、mentor / adviser の両方で upcoming_pair_works が表示されることを view か system spec で1本押さえておくと再発防止になります。

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@app/views/home/_mentor_dashboard.html.slim` around lines 28 - 29,
追加された表示分岐(@upcoming_pair_works
をレンダリングする部分)がロール別ダッシュボードで漏れる不具合の元なので、ビューではなくテストで保証します:新しい system spec(例:
Dashboard as mentor and adviser)を追加して、ログインユーザーの role を mentor と adviser
に切り替えたそれぞれのケースでページを訪問し、部分テンプレート '_upcoming_pair_works' またはページ内に
`@upcoming_pair_works` に由来する固有のテキスト/セレクタが存在することを期待(have_selector / have_text
等)するアサーションを追加してください。テストは `@upcoming_pair_works` を事前に作成しておき、render
'upcoming_pair_works' を呼ぶ
view(_mentor_dashboard.html.slim)を対象に両ロールで表示されることを確認するようにします。
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@app/views/home/_mentor_dashboard.html.slim`:
- Around line 28-29: 追加された表示分岐(@upcoming_pair_works
をレンダリングする部分)がロール別ダッシュボードで漏れる不具合の元なので、ビューではなくテストで保証します:新しい system spec(例:
Dashboard as mentor and adviser)を追加して、ログインユーザーの role を mentor と adviser
に切り替えたそれぞれのケースでページを訪問し、部分テンプレート '_upcoming_pair_works' またはページ内に
`@upcoming_pair_works` に由来する固有のテキスト/セレクタが存在することを期待(have_selector / have_text
等)するアサーションを追加してください。テストは `@upcoming_pair_works` を事前に作成しておき、render
'upcoming_pair_works' を呼ぶ
view(_mentor_dashboard.html.slim)を対象に両ロールで表示されることを確認するようにします。

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ba872272-a3c9-4e3e-8f9d-4cd888b3a326

📥 Commits

Reviewing files that changed from the base of the PR and between e848542 and 9c4a267.

📒 Files selected for processing (3)
  • app/views/home/_adviser_dashboard.html.slim
  • app/views/home/_mentor_dashboard.html.slim
  • db/fixtures/pair_works.yml

@mousu-a
Copy link
Copy Markdown
Contributor Author

mousu-a commented Mar 7, 2026

@machida
お疲れ様です!
ペアワーク機能にて、近日開催のペアワークが受講生のダッシュボードにしか表示されていなかったのでメンター、アドバイザーのダッシュボードでも表示されるようにしました。
こちらデザインの確認をお願いしたいです🙏

受講生

image

メンター、アドバイザー

メンター
スクリーンショット 2026-03-07 10 01 14

アドバイザー
スクリーンショット 2026-03-07 9 29 45

見てほしいところ

ウィンドウ幅が1026pxを超えると、↑のキャプチャのように下部に空白ができてしまうようです。

それ以下であればこの空白は発生しません。
スクリーンショット 2026-03-07 10 01 20


現在メンターのダッシュボードのみ確認できているのですが、ウィンドウ幅を広げすぎるとこんな感じになってしまうようです。
(アドバイザー、受講生も構造的には同じだと思いますが、物理的になることがないという感じです
スクリーンショット 2026-03-07 10 01 07

動作確認について

変更確認方法を試していただければ一通り確認できるようになっています🙏

@machida
Copy link
Copy Markdown
Member

machida commented Mar 7, 2026

@mousu-a ありがとうございます!!確認します💪

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.

2 participants