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

Ability to override OpenMRS frontend core version based on frontend customizations file #79

Merged
merged 2 commits into from
Aug 1, 2024

Conversation

corneliouzbett
Copy link
Contributor

This PR updates the OpenMRS Frontend Core version used in the build process to match the version specified in the frontend customizations file, if present.

Copy link
Contributor

@rbuisson rbuisson left a comment

Choose a reason for hiding this comment

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

@corneliouzbett , should we actually set the coreVersion as part of Ozone?
It seems that as such O3 doesn't work as it should.

Co-authored-by: Romain Buisson <rbuisson@users.noreply.github.com>
@corneliouzbett
Copy link
Contributor Author

corneliouzbett commented Jul 8, 2024

Should we actually set the coreVersion as part of Ozone?

It makes sense to me why we should tie "coreVersion" with an Ozone release. I believe that providing a mechanism to override, along with a default setting (use bleeding edge next and for Ozone release, changing it to a stable coreVersion), should suffice for now.

This is an O3 issue rather than Ozone.

@corneliouzbett corneliouzbett changed the title Update OpenMRS Core version based on frontend customizations Override OpenMRS frontend core version based on frontend customizations file Jul 8, 2024
@rbuisson rbuisson changed the title Override OpenMRS frontend core version based on frontend customizations file Ability to override OpenMRS frontend core version based on frontend customizations file Jul 8, 2024
@corneliouzbett corneliouzbett merged commit 05af7ae into main Aug 1, 2024
2 checks passed
@corneliouzbett corneliouzbett deleted the chores branch August 1, 2024 09:03
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.

2 participants