Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add volunteer interest to rsvp #481

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cjmanetta
Copy link

This PR adds a volunteer interest checkbox to the Rsvp form and migrates the Rsvp table to contain a volunteer interest boolean. This can be viewed in the organizer attendee details panel.
#443
PTAL

@lilliealbert
Copy link
Member

@cjmanetta Thank you for this! Overall it looks really good. @tjgrathwell and I were chatting about whether it makes more sense for volunteer interest to live on the RSVP model, or if there should be some kind of join table between chapters & users like ChapterVolunteerInterest or something.

The main concern is that we'd like to collect this info when people RSVP, but it isn't really related to the RSVP itself — when we show the responses, we'll probably do it on a per-chapter basis, aggregating all the people who want to help out, and thinking about that query highlights for me how it'd be a little weird to get the info off of RSVP.

What do you think? Do you think a join table would be reasonable, or do you see another way we could model this? 😸

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