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.

Sign up for GitHub

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

Jump to bottom

Clinic: adds API endpoints and UI to list, search and view patients #3

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Clinic: adds API endpoints and UI to list, search and view patients #3

wants to merge 14 commits into from

Conversation

ashutoshgngwr
Copy link
Member

@ashutoshgngwr ashutoshgngwr commented Oct 26, 2023 •
edited
Loading

Also piggybacking some minor refactors.

TODO:

  • Fix remote API call handling: remote API requests trigger when Reagent render is invoked, and it is not ideal for scenarios where re-render doesn't change request parameters.

@ashutoshgngwr
clinic: add a more complete generator for date strings
17eef5f
@ashutoshgngwr
clinic: make phone a required field in create patient form
ade32b4
@ashutoshgngwr
clinic: add stricter validation for phone number inputs
3f78bdd
@ashutoshgngwr
clinic: add api endpoint to search patients by phone number
110044c
@ashutoshgngwr
clinic: add api endpoint to lookup patients by their id
688fc17
@ashutoshgngwr
clinic: add ui components for viewing patient details
e5c84da
@ashutoshgngwr
clinic: add ui to list patients
e05fd55
@ashutoshgngwr
clinic: use r/props for all components
b1f1f6a 
Because:
1. React components accept an OPTIONAL map of props as their first arguement.
2. If the first arg is not a map, it is considered a child.
3. Therefore, map destructuring the first arg won't work.
4. We can still use non-map props, e.g. `[component prop-1 prop-2 ...]`, but
   this isn't idea for components that want to accept children. Therefore,
   sticking to the optional-map-style props is consistent way to declare
   components.
@ashutoshgngwr
clinic: simplify state management for create patient form
be45aa8
@ashutoshgngwr
clinic: add search field to patient list ui
5b5121c
@ashutoshgngwr ashutoshgngwr changed the title Adds API endpoints and UI to list, search and view patients Clinic: adds API endpoints and UI to list, search and view patients Oct 26, 2023
@ashutoshgngwr ashutoshgngwr force-pushed the clinic/feat/lookup-patients branch from 395ceed to 5063ef1 Compare October 27, 2023 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ashutoshgngwr