Conversation
マイグレーション、モデル、フォームまで実装完了。 表示ロジックとテストは次のコミットで追加予定。 - マイグレーション: Team/Eventにブラインド設定カラム追加 - Event#organizer_visible_to?, #participants_visible_to? メソッド追加 - チーム設定画面にデフォルト設定を追加 - イベントフォームに個別設定(ラジオボタン)を追加 - コントローラーでパラメータ許可 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- 主催者表示: 非表示時は「(非表示)」、主催者本人には「(あなた)」表示 - 参加者数: 非表示時は定員のみ表示、満席時は「(満席)」表示 - 参加者リスト: 非表示時は「参加者リストは非表示です」メッセージ表示 - 管理者には常にすべて表示、主催者本人には自分の主催者ステータス表示 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- イベント一覧・マイページでのブラインド表示対応 - 管理者権限での表示時に注釈を追加 - Event#organizer_visible_to?のnil安全性を修正 - 包括的なモデルテストを追加(92テスト全てパス) 変更内容: - イベント詳細:主催者・参加者に「(管理者権限で表示中)」注釈 - イベント一覧:ブラインド設定時は「非表示」と表示 - マイページ:主催イベントは「(あなた)」表示、参加イベントはブラインド設定に従う - モデルテスト:チーム設定/イベント設定/権限による全パターンをカバー 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- イベント詳細ページでのブラインド表示テスト(8テスト) - 管理者・主催者・一般メンバーの各視点での表示確認 - チーム設定のデフォルト設定変更テスト - デフォルト設定が新規イベントに反映されることの確認 全テストパス(モデル92 + システム8 = 100テスト) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Capybara/NegationMatcherの自動修正 - RSpec/NestedGroupsの上限を5に設定 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! このプルリクエストは、イベントの主催者名と参加者リストの表示を柔軟に制御できる「ブラインド機能」を導入します。これにより、参加者が「誰が主催・参加するか」ではなく「イベント内容」に基づいて参加を判断できるようになります。チーム全体でデフォルト設定を管理しつつ、各イベントで個別に表示設定を上書きできるため、運用の柔軟性が向上します。管理者や主催者には特別な表示ルールが適用され、情報へのアクセス権限が適切に管理されます。 Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request introduces a new "blind" feature for events, allowing teams to control the visibility of event organizers and participant lists. The changes include adding new boolean columns to the teams and events tables for default team settings and per-event overrides, updating controllers to permit these new parameters, and modifying various view files to conditionally display organizer names and participant counts/lists based on these settings and the viewing user's role (admins and event organizers always see full details). Comprehensive model and system tests were added to validate the new visibility logic, alongside a minor Rubocop configuration update. A review comment suggests that when the participant list is hidden, the event capacity (currently still displayed) should also be concealed to fully align with the feature's stated goal of hiding all participant-related information.
| <% if event.full? %> | ||
| <%= capacity_text(event) %>(満席) | ||
| <% else %> | ||
| <%= capacity_text(event) %> | ||
| <% end %> |
- 参加者リストが非表示の場合、定員も非表示に - 満席の場合のみ「満席」と表示(参加不可を明示) - システムテストを追加して動作確認 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
イベント作成時に参加者ブラインド設定を明示的に「表示する」 に設定することで、定員が表示されることをテスト 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Summary
主な変更内容
データベース
teamsテーブルにhide_organizer_by_default,hide_participants_by_defaultカラム追加eventsテーブルにhide_organizer,hide_participantsカラム追加(nil = チーム設定に従う)モデル
Event#organizer_visible_to?(membership)- 主催者表示判定メソッドEvent#participants_visible_to?(membership)- 参加者表示判定メソッドビュー
権限設定
テスト
動作確認項目
🤖 Generated with Claude Code