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.

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) add support to search relationship by patient identifier on registration form #851

Conversation

Herobiam
Copy link

Requirements

Summary

Enhancing the registration form's relationship creation feature has been addressed in this pull request (PR). Formerly, the search functionality was limited to first and last names, omitting the option to search using the patient identifier. The proposed changes in this PR extend the search capabilities to encompass patient identifiers, thereby providing a more comprehensive and efficient method for establishing relationships within the registration process.

Screenshots

Screen.Recording.2023-10-16.at.4.56.58.PM.mov

Related Issue

None.

Other

None.

@Herobiam Herobiam force-pushed the feat/allow-relationship-search-by-patient-identifier branch from 2df0a0c to 13dd701 Compare October 17, 2023 07:56
@Herobiam Herobiam requested a review from pirupius October 18, 2023 13:26
const patientEndpoint = `${BASE_API}/patient?q=${query}`;
return openmrsFetch(patientEndpoint, { signal });
} catch (error) {
console.error('Error fetching person:', error);
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Herobiam can you remove this console log please

@ibacher
Copy link
Member

ibacher commented Oct 25, 2023

I’m closing this in favour of #856, which implements the same logic.

@Herobiam
Copy link
Author

Okay great thanks @ibacher

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants