Skip to content

Allow co-organizers to preview attendee list - #172

Merged
malkoG merged 4 commits into
devfrom
feature/attendee-list-preview
Apr 8, 2026
Merged

malkoG merged 4 commits into
devfrom
feature/attendee-list-preview

Conversation

@malkoG

@malkoG malkoG commented Apr 8, 2026 •

Copy link
Copy Markdown
Member

Summary

Co-organizers listed in eventOrganizers can now view the attendee list on the event detail page. The access check matches co-organizers via both actors.userId and userFediverseAccounts to handle remote actors without a local userId. Also fixes text overflow issues in the attendee card and registration question labels.

Test plan

  • Add a co-organizer to an event, verify they can see the attendee list
  • Test with a co-organizer whose actor is a remote fediverse account (no userId)
  • Verify long question labels wrap correctly in the registration form
  • Verify long Q&A text in attendee cards doesn't overflow

Summary by CodeRabbit

Release Notes

  • Bug Fixes

    • Enhanced attendee list UI with improved text handling for long names and content, preventing layout overflow while maintaining visual stability
    • Improved text wrapping and selection behavior in event registration form questions for better content display
  • New Features

    • Event owners and co-organizers now have access permissions to view attendee information and details

malkoG added 4 commits April 9, 2026 04:50
Check eventOrganizers table in addition to event
owner and group members for attendee list access.
Add break-all and block display to prevent long question
text from overflowing. Make label text selectable.
Match co-organizers via userFediverseAccounts when
the remote actor has no userId set in the actors table.
Add break-all on Q&A text, truncate long names,
and overflow-hidden on card container.
@malkoG
malkoG merged commit 0ca9b91 into dev Apr 8, 2026
1 check failed
@coderabbitai

coderabbitai Bot commented Apr 8, 2026 •

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 59828a31-8891-45c1-b400-d2a6f71efaf1

📥 Commits

Reviewing files that changed from the base of the PR and between df4b285 and 6f6cbf0.

📒 Files selected for processing (3)
  • src/routes/events/$eventId/index.tsx
  • src/routes/events/$eventId/register.tsx
  • src/server/controllers/events/attendees.ts

📝 Walkthrough

Walkthrough

Updated attendee and question display UI to handle long text with overflow management, and expanded authorization logic for attendee list access to check event ownership, group membership, and co-organizer status across federated accounts.

Changes

Cohort / File(s) Summary
Attendee List Display
src/routes/events/$eventId/index.tsx
Added overflow-hidden, min-w-0, truncate, and shrink-0 classes to attendee cards for proper text handling and layout stability. Changed question/answer rendering to use break-all word breaking.
Registration Questions UI
src/routes/events/$eventId/register.tsx
Applied overflow-hidden to container and enhanced question Label with block, break-all, and select-text classes for improved text wrapping and selectability.
Attendee Authorization
src/server/controllers/events/attendees.ts
Refactored access control from single-check logic to composite authorization checking event ownership, group membership, and co-organizer status via federated accounts. Now queries organizerId and joins eventOrganizers table with federated account matching before returning 403.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 Hop, skip, and text override,
No more squishing! Words now hide,
Who can peek? Check the crew—
Owner, groups, and orgs too! ✨

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/attendee-list-preview

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.

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.

1 participant