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

(Upgrade): Update the esm-patient-common-lib version. #1846

Merged
merged 4 commits into from
May 16, 2024

Conversation

hadijahkyampeire
Copy link
Contributor

Requirements

  • This PR has a title that briefly describes the work done including the ticket number. If there is a ticket, make sure your PR title includes the ticket number in the format OHRI-123 My PR title.
  • My work includes tests or is validated by existing tests.

Summary

  • This PR updates the openmrs and common-lib versions, this necessitated some changes since some workspace functions were moved to other places or are no longer being exported. But this fixes the workspace bugs we have been seeing.
  • This PR also updates the openmrs and common lib versions inside the ohri-esm-commons-lib.

Screenshots

Uploading Workspaces-fix.mov…

Related Issue

  • No ticket

Other

@hadijahkyampeire hadijahkyampeire requested review from arodidev, ebambo and VeronicaMukuhi and removed request for VeronicaMukuhi May 16, 2024 11:07
@denniskigen
Copy link

@hadijahkyampeire please run yarn again and commit the updated lockfile.

alaboso
alaboso previously approved these changes May 16, 2024
packages/esm-commons-lib/src/index.ts Outdated Show resolved Hide resolved
Comment on lines +42 to +43
"@openmrs/esm-framework": "*",
"@openmrs/esm-patient-common-lib": "*",

Choose a reason for hiding this comment

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

These should be X.x versions, right @ibacher?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, although I don't know that that's a hard requirement.

@eudson eudson merged commit c50c7a3 into dev May 16, 2024
5 checks passed
@eudson eudson deleted the hk-update-openmrs-esm-common-lib branch May 16, 2024 12:11
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.

5 participants