Skip to content

Add SCEvents visibility and edit-access handling#2089

Merged
trista-chen-29 merged 1 commit intodevfrom
trista/event_visibility
Apr 19, 2026
Merged

Add SCEvents visibility and edit-access handling#2089
trista-chen-29 merged 1 commit intodevfrom
trista/event_visibility

Conversation

@trista-chen-29
Copy link
Copy Markdown
Collaborator

Closes #2082

Changes

  • Added status, visibility, and minimum_visible_role fields to:
    • Create Event page
    • Edit Event page
  • Added frontend filtering for SCEvents event cards based on:
    • draft vs published
    • public vs private
    • minimum visible role
  • Kept edit access aligned with the current SCEvents backend:
    • only event admins can edit events

Current visibility behavior

User type Published + Public Published + Private (member) Published + Private (officer) Draft
Non-member
Member
Officer (not event admin)
Event admin
Admin

Current edit behavior

User type Can edit event
Non-member
Member
Officer (not event admin)
Event admin
Admin (not event admin)

Verification

  • feature flag OFF
Screenshot 2026-04-18 at 2 31 16 AM Screenshot 2026-04-18 at 2 31 06 AM
  • feature flag ON
    • Public / Non-member
      Only public published events are visible
Screenshot 2026-04-18 at 2 21 19 AM
  • Member
Screenshot 2026-04-18 at 2 28 43 AM
  • Officer (event creator)
    Draft event is visible and editable
Screenshot 2026-04-18 at 2 21 43 AM
  • Another officer (not event admin)
Screenshot 2026-04-18 at 2 22 15 AM
  • Admin
Screenshot 2026-04-18 at 2 23 22 AM Draft event is visible, but edit is restricted to event admins Screenshot 2026-04-18 at 2 23 29 AM

Copy link
Copy Markdown
Collaborator

@steeevin88 steeevin88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome stuff

@trista-chen-29 trista-chen-29 merged commit eee79c1 into dev Apr 19, 2026
4 checks passed
@trista-chen-29 trista-chen-29 deleted the trista/event_visibility branch April 19, 2026 03:17
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.

Create frontend logic for event visibilities

3 participants