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

Refactor location view details. #1351

Merged
merged 20 commits into from
Mar 26, 2024
Merged

Refactor location view details. #1351

merged 20 commits into from
Mar 26, 2024

Conversation

peterMuriuki
Copy link
Collaborator

@peterMuriuki peterMuriuki commented Mar 20, 2024

closes #1339
closes #1341

Changes included with this PR

  • Adds view for location view details that will be used to also show inventory.
  • Adds new package to house our in house fhir definitions and utils.
  • Replaces the previous LocationUnitDetail component.
  • Style tweak on the keyValue pair render component -> cause quite a few snapshots to change

Checklist

  • tests are included and passing
  • documentation is changed or added
  • Internationalization:
    • Ensure all strings are internationalized.
  • Role-Based Access:
    • Verify that all user actions have appropriate role-based access permissions set.

@peterMuriuki peterMuriuki changed the title 1339 location view details Refactor location view details. Mar 25, 2024
@peterMuriuki peterMuriuki marked this pull request as ready for review March 26, 2024 07:55
@peterMuriuki peterMuriuki merged commit 07c9026 into main Mar 26, 2024
0 of 2 checks passed
@peterMuriuki peterMuriuki deleted the 1339-location-view-details branch March 26, 2024 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants