Conversation
Walkthroughレポート一覧ビューに、 Changes
Sequence Diagram(s)sequenceDiagram
actor Mentor as メンター
participant Controller as ReportsController#index
participant View as index.html.slim
participant Partial as _unconfirmed_links_open
Mentor->>Controller: GET /reports
Controller-->>View: render reports (@reports) & pagination
alt mentor_login? && params[:unchecked]
View->>Partial: render _unconfirmed_links_open("未チェックの日報を一括で開く")
Partial-->>View: partial HTML
end
View-->>Mentor: HTML response
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
app/views/reports/index.html.slim (2)
48-49: 未チェックタブでのみ表示するよう条件を追加してください現状だと「全て」タブでもボタンが出ます。文言と機能の整合のため、未チェック表示時に限定しましょう。
- - if mentor_login? + - if mentor_login? && params[:unchecked] = render partial: 'unconfirmed_links_open', locals: { label: '未チェックの日報を一括で開く' }
49-49: 文言のi18n化（任意）テスト容易性と今後の多言語化に備え、
locals: { label: t('.open_unchecked_reports_bulk') }などに移行を検討ください。
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 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
🔇 Additional comments (1)
app/views/reports/index.html.slim (1)
48-49: 権限要件の確認（admin も対象なら条件を合わせる）上部のタブ表示は
admin_or_mentor_login?ベースですが、ボタンはmentor_login?のみです。管理者にも許可する仕様なら下記に変更してください。- - if mentor_login? && params[:unchecked] + - if admin_or_mentor_login? && params[:unchecked]
|
@yokomaru さん |
|
@sharoa119 |
sharoa119
requested review from
yokomaru
and removed request for
okuramasafumi
|
お疲れ様です☕️ ざっとテキストで共有させていただきますが、言葉で伝えづらい部分もあり、Discordで繋ぎながら確認することも可能なので気軽にご連絡ください🙌 自分が理解したことについての整理
以前の仕様
前回のsharoaさんの修正
今回の対応
現状の挙動について
全ての日報ページ(unchecked: false or nil)での挙動
2025-09-18.15.13.58.mov
2025-09-18.15.30.45.mov
2025-09-18.15.31.37.mov未チェック日報一覧(unchecked: true)
2025-09-18.15.16.10.mov確認したいこと
余談
ブラウザのポップアップのブロックについて
ものすごく長文になってしまい＆読みづらく申し訳ありません・・🙇♀️ |
|
@yokomaru さん
まずこちらについてですが、こちらは私のコードミスです。
こちらについて、私の認識が間違っていました💦
が適応されるので、ページネーションの上になることになります。 一応町田さんに確認しますね！
こちらは言われて、確かに！となりました！ 少々ご対応をお待ちください🙇♀️ |
|
@sharoa119 上記承知いたしました〜！よろしくお願いしますっ！ |
|
@machida さん つまり、昨日おっしゃっていた
が適応されるので、ボタンの位置はページネーションの上になると思います。 再度確認のため、こちらで問題ないかご教示いただければと思います。 誤認識すみません😭 |
|
@machida さん 色々と考えてたら なので２通りやってみました。 上部のページャの上 下部のページャの上 
どちらが良いでしょうか。 |
|
そのページに表示されている分だけを一括で表示、了解ですー。今気付いて良かったです👍 下部のページャの上でお願いします🙏設置したらデザインを調整するのでメンションお願いしますー |
|
@machida さん |
|
@sharoa119 (CC: @machida ) すみません、こちらこの機能がないので未確認の日報が大量に貯まる状態になってしまっているので緊急でリリースさせていただければと思います。 僕の方でこちらの内容でhotfixブランチを作ってリリースしてしまいたいと思います。 |
下部ページネーションの上に配置
komagata
force-pushed
the
bug/reenable-bulk-open-unread-reports
branch
from
e43e2e3 to
0fa75f5
Compare
|
#9183 (comment) |
4 participants
Issue
概要
PR:#8976 で削除してしまった
「未チェックの日報を一括で開く」ボタンを復活させる
変更確認方法
bug/reenable-bulk-open-unread-reportsをローカルに取り込むforeman start -f Procfile.devでローカル環境を立ち上げるkomagataでログインする日報タブを選択し、その中の未チェックタブを選択した際に下部に表示される「未チェックの日報を一括で開く」ボタンを確認。（ボタンをクリックして別タブでリンクが開くことも確認）動作確認時の注意
「未チェックの日報を一括で開く」ボタンを押した際、先頭の1件しか開かない場合があります。
これは Chrome のポップアップブロック機能（
window.openを利用したタブの同時オープンも対象）が原因です。動作確認を行う場合は、以下いずれかでポップアップを許可してください。
Screenshot
変更前
変更後
Summary by CodeRabbit