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

FLAG-73: upgrade openmrs version from 1.9.9 to 2.0.0 #58

Merged
merged 1 commit into from
Jun 14, 2024

Conversation

ManojLL
Copy link
Contributor

@ManojLL ManojLL commented Jun 10, 2024

Description of what I change

Upgrade the openMRS platform version from 1.9.9 to 2.0.0

Issue ### Worked On
worked on FLAG-73

@ManojLL ManojLL marked this pull request as ready for review June 10, 2024 17:11
pom.xml Outdated
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
<uiframeworkVersion>3.11.0</uiframeworkVersion>
<metadatadeployVersion>1.8.1</metadatadeployVersion>
<metadatasharingVersion>1.2.2</metadatasharingVersion>
<webservices.rest-omod.version>2.40.0</webservices.rest-omod.version>
<legacyuiVersion>1.2.3</legacyuiVersion>
<omodHbmConfig>
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you explain why you added this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we had a profile for platform 2.0.0, in there contains these properties for platform 2.0.0 so just add them to here.

Copy link
Contributor

Choose a reason for hiding this comment

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

Do we get any errors if we don't include these?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

module build successfully without this properties.

omod/pom.xml Outdated
<artifactId>legacyui-omod</artifactId>
</dependency>
<dependency>
<groupId>javax.servlet</groupId>
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need this dependency?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, without adding these dependencies we can not use legacy ui things in controllers.

Copy link
Contributor

Choose a reason for hiding this comment

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

Do we get any errors without "javax.servlet-api" ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we can remove "javax.servlet-api" and I updated the changes

@@ -107,18 +107,31 @@
<version>${metadatasharingVersion}</version>
<scope>test</scope>
</dependency>
<dependency>
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need both legacyui-omod and legacyui-api?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I removed legacyui-api dependency

pom.xml Outdated
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
<uiframeworkVersion>3.11.0</uiframeworkVersion>
<metadatadeployVersion>1.8.1</metadatadeployVersion>
<metadatasharingVersion>1.2.2</metadatasharingVersion>
<webservices.rest-omod.version>2.40.0</webservices.rest-omod.version>
<legacyuiVersion>1.2.3</legacyuiVersion>
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you try using a more recent version of legacyui?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I updated it to latest legacy UI version

FLAG-73: remove unwanted dependencies and properties

FLAG-73: remove unwanted dependencies

FLAG-73: update legacy UI to latest version
@wikumChamith wikumChamith merged commit 82746c0 into openmrs:master Jun 14, 2024
3 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
2 participants