-
Notifications
You must be signed in to change notification settings - Fork 479
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
User (Doctor) Autocomplete for Consultation Verified By #6194
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@rithviknishad modify the cypress test |
@nihal467 cypress will keep failing here until backend is merged |
👋 Hi, @rithviknishad, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
👋 Hi, @rithviknishad, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
LGTM |
@rithviknishad We truly appreciate your efforts. Thank you for taking the time to contribute; this is a very valuable contribution to us 🥇. We always welcome your contribution 🙂, so feel free to contribute to anything anytime, and never lose that spirit of innovation 🙌. |
WHAT
🤖 Generated by Copilot at c8d2e64
The pull request adds a user autocomplete field for selecting the verifying doctor in a consultation form and displays the verifying doctor's information in the consultation details. It also refactors the user types and improves the user filtering logic. The affected files are
ConsultationForm.tsx
,models.tsx
,UserAutocompleteFormField.tsx
, andConsultationDetails.tsx
.Required Backends:
Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist
HOW
🤖 Generated by Copilot at c8d2e64
verified_by_object
property toConsultationModel
andFormDetails
to store the user object of the verifying doctor (link, link)ConsultationDetails
usingverified_by_object
ordeprecated_verified_by
as a fallback (link)TextAreaFormField
withUserAutocompleteFormField
for theverified_by
field inConsultationForm
to allow selecting a user from a filtered list (link)search_filter
andquery
parameters togetFacilityUsers
inUserAutocompleteFormField
to filter the facility users by the search text and other criteria (link)ConsultationForm
and update theverified_by
andverified_by_object
properties accordingly (link, link, link)verified_by
property from theset_form
action of thedispatch
function inConsultationForm
and add theverified_by
andverified_by_object
properties instead (link, link)UserBareMinimum
type to represent the minimal user object with id and username and use it as a base type for other user types and as a property type for other models (link, link, link, link, link)label
prop to theUserAutocompleteFormField
component for theassigned_to
field inConsultationForm
to specify the label text for the field (link)