-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat: User accessibility information input #353
base: develop
Are you sure you want to change the base?
Conversation
767c7cc
to
282fd79
Compare
282fd79
to
cb146ac
Compare
@click=" | ||
() => { | ||
booking.accessibilityInfo = newAccessibility; | ||
updateAPI(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just double-checking it is the case on the back-end too; will the prod team be emailed about an update made to accessibility info after the booking has been already confirmed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And if accessibility info is removed (e.g. the person in the wheelchair is no longer coming, and their seat is taken by somebody else in the family instead), will that also be emailed to prod?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a great catch, and led me to a further investigative chain. I've pulled the PR back to draft, since it's going to need some further API work to go with it to make sure that emails are sent properly.
All looks pretty good - just a small number of minor tweaks (or not, just questions), and then good to go |
4642757
to
2740efa
Compare
Implementing
accessibilityInfo
, allowing users to input any access(ability) requirements and admins/box office to view it (resolves #320)