-
Notifications
You must be signed in to change notification settings - Fork 10
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
[HIE-2] Skeleton code for client registry search flow #5
Conversation
api/src/main/java/org/openmrs/module/clientregistry/api/model/CRPatient.java
Outdated
Show resolved
Hide resolved
api/src/main/java/org/openmrs/module/clientregistry/api/providers/FhirCRConstants.java
Outdated
Show resolved
Hide resolved
|
||
@Search | ||
public List<CRPatient> searchClientRegistryPatients( | ||
@OptionalParam(name = Patient.SP_NAME) StringAndListParam name, |
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.
leave this unimplemented for now, will implement in separate ticket
...n/java/org/openmrs/module/clientregistry/api/providers/r4/CRPatientFhirResourceProvider.java
Outdated
Show resolved
Hide resolved
...n/java/org/openmrs/module/clientregistry/api/providers/r4/CRPatientFhirResourceProvider.java
Outdated
Show resolved
Hide resolved
* the client registry | ||
* @throws IllegalArgumentException if defined transaction type is unsupported | ||
*/ | ||
public CRPatientService getPatientService() throws IllegalArgumentException { |
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 only needs to be determined once on module startup (and on property changed/deleted), maybe we move the logic into ClientRegistryActivator
?
api/src/main/java/org/openmrs/module/clientregistry/api/model/CRPatient.java
Outdated
Show resolved
Hide resolved
[HIE-2] define ihe-pix operation and start implementation of getCRPatient [HIE-2] adding config options for default system and get patient endpoint [HIE-2] drop format param and change targetSystems to OR [HIE-2] move targetSystem param logic to controller [HIE-2] change to OperationParam in getCRPatient provider method cleanup, adding comments, refactoring Patient parsing
api/src/main/java/org/openmrs/module/clientregistry/api/impl/FhirCRPatientServiceImpl.java
Outdated
Show resolved
Hide resolved
… ihe-pix CR endpoint
Closing PR, moved to #11 |
wip