-
Notifications
You must be signed in to change notification settings - Fork 4
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
[Closes #97, #44] Admin/Staff/Volunteer user can register a new Patient from an unused QR code #109
Conversation
I've incorporated the feedback from Weds night when I demoed it, but let me know if I missed anything else! I think any other major form changes can be moved to a new PR so we can use this registration form to start user testing with
I've reduced the strictness regarding which fields are required for form validation to only the information needed for Basic Information for now
I made adjustments to these inputs by adding a
I've added a
Fixed the conditional logic check to prevent this!
Made some adjustments to the form to prevent |
To reduce the number of API calls, api calls for a search query are debounced
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.
Ok, this is a great starting point, there's more than enough work here that I'm going to merge it in, and then we can start creating new issues around adjustments/fixes/enhancements...
Closes #97, closes #44.
This PR adds the ability for an authorized user to register a new patient after loading a URL tied to a new patient ID.
useForm
hook for form validation andAccordion
component to reduce the screen height of the formComboBox
componentpatientId
that already exists (either by refreshing the page or resisting the URL again), the preexisting information for the patient will be the initial values of the formSome things to note:
PATCH
requestContact
record will still exist in the database but will be unlinked from the patient