Skip to content

Decline feature#360

Merged
rylany27 merged 4 commits into
mainfrom
decline_feature
Apr 7, 2026
Merged

Decline feature#360
rylany27 merged 4 commits into
mainfrom
decline_feature

Conversation

@Invictus108
Copy link
Copy Markdown
Collaborator

Adds a decline button so users can indicate if they don't want to go to the event

This will later be used for email reminders and invitations

It will modify the Firebase schema as it goes, adding a isdeclined boolean field

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 5, 2026

Visit the preview URL for this PR (updated for commit db1e431):

https://studybuddy-392c7--pr-360-6gdofp79.web.app

(expires Tue, 14 Apr 2026 01:14:42 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 1e823c805395bb7b968e98b8401ce59756126c01

@rylany27
Copy link
Copy Markdown
Collaborator

rylany27 commented Apr 6, 2026

A few things:

  1. An admin shouldn't have the option to decline an invitation
  2. Only the admin should be able to see who's declined (for now, makes the rest easier)
  3. Once a participant declines, they should be redirected to the landing page, and the event should be removed from that user's firebase (they should still be able to access the same event via the dashboard link as before)
  4. On the admin's side, if a declined participant rejoins the event and fills out their availability, the declined status should be removed
  5. There should be a confirm to decline popup (feel free to reuse the confirm to delete event one)

@Invictus108
Copy link
Copy Markdown
Collaborator Author

Ok I think I got all of the changes. Let me know if there is anything else

@rylany27
Copy link
Copy Markdown
Collaborator

rylany27 commented Apr 6, 2026

Looks good I think. One more thing I ran into is that there's a Firebase error when trying to decline an invitation as a guest. I think it'd be simplest to just remove the option entirely for guests.

(it might be worth discussing in the future phasing out guest accounts entirely)

@Invictus108
Copy link
Copy Markdown
Collaborator Author

sorry about that. Fixed it

@rylany27 rylany27 merged commit 25cd081 into main Apr 7, 2026
2 checks passed
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