Skip to content

HTMLWIDGETS-26: Migrate to Platfrom 2.7.0 and Support Java 21#9

Merged
dkayiwa merged 2 commits intoopenmrs:masterfrom
wikumChamith:HTMLWIDGETS-26
May 19, 2025
Merged

HTMLWIDGETS-26: Migrate to Platfrom 2.7.0 and Support Java 21#9
dkayiwa merged 2 commits intoopenmrs:masterfrom
wikumChamith:HTMLWIDGETS-26

Conversation

@wikumChamith
Copy link
Copy Markdown
Member

package org.openmrs.module.htmlwidgets.web.demo;

public enum PatientLocationMethod {
EARLIEST_ENCOUNTER,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What is this used for?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This enum has been moved to the reportingcompatibility module. Instead of introducing a new dependency on that module, I decided to copy the enum directly into this module.

PatientSetService.java#L579-L581

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is it used anywhere apart from tests?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yeah, it is only used in the Demo class, so I decided to move the enum to that class.

@wikumChamith wikumChamith force-pushed the HTMLWIDGETS-26 branch 2 times, most recently from 5aea694 to 947ae7f Compare May 19, 2025 06:31
@dkayiwa
Copy link
Copy Markdown
Member

dkayiwa commented May 19, 2025

Are you also going to create a branch and bump the module version?

@wikumChamith
Copy link
Copy Markdown
Member Author

@dkayiwa dkayiwa merged commit 27ac129 into openmrs:master May 19, 2025
5 checks passed
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