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: KHP3 follow up changes on the providers: Filters, UI #552

Merged
merged 8 commits into from
Jan 17, 2025

Conversation

its-kios09
Copy link
Collaborator

Requirements

  • This PR has a title that briefly describes the work done, including the ticket number if there is a ticket.
  • My work conforms to the OpenMRS 3.0 Styleguide.
  • I checked for feature overlap with existing widgets.

Summary

I needed to add a comment on the provider attribute for the HIE FHIR bundle reference being persisted. It is breaking because the omod I have is missing the following constant: public static final String PROVIDER_HIE_FHIR_REFERENCE = "67b94e8e-4d61-4810-b0f1-d86497f6e553";
Additionally, based on suggestions from today’s meeting, the following fields should be set to read-only: license, National ID, registration number, and expiry date to prevent them from being modified.

Screenshots

PR

unedittable

None.

Related Issue

None.

Other

None.

@its-kios09 its-kios09 enabled auto-merge (squash) January 16, 2025 20:52
@its-kios09
Copy link
Collaborator Author

@donaldkibet @patryllus @ojwanganto any reactions??

HWR

@donaldkibet donaldkibet merged commit 57d4e5e into palladiumkenya:main Jan 17, 2025
1 check passed
@ojwanganto
Copy link
Contributor

Thanks, @its-kios09. This looks really nice. Is it possible to move from the expanded section to the table? It will look better in that point

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.

3 participants