Dietary restrictions #2155
Open
Dietary restrictions #2155
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This change will add a column to the Member model, allowing users to provide dietary information. This should support Codebar hosts to provide good food options for all attendees
biggianteye
reviewed
Dec 22, 2024
This change adds a simple input to the form that collects Member information. I've included a translation in English and in French, because that happens to be another language I speak!
jonodrew
force-pushed
the
dietary-restrictions
branch
from
03ebc5e to
2c2a167
Compare
|
I can see there's a failing test here, but I don't think there's anything I can do about it - unless there are some credentials I'm missing? I can't add |
|
Hey @jonodrew thank you so much for this PR, we really appreciate you putting in the time to help us. However, we envisioned it as a checklist of options : Could this change be implemented before we merge please? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This change should:
this should close #1842