Skip to content

Update application.properties #187

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Update application.properties #187

wants to merge 1 commit into from

Conversation

nsundara
Copy link
Contributor

Hi Pablo,

Can you check the property name changes for UCP? I thought, this was fixed but, it was still using ucp and not oracleucp.
@nsundara nsundara requested a review from psilberk January 19, 2022 17:28
psilberk
psilberk suggested changes Jan 19, 2022
View reviewed changes
Copy link
Contributor

@psilberk psilberk left a comment

Choose a reason for hiding this comment

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

Hi Nirmala, in order for this to work with the newer values (...oracleucp...) you have to use a SpringBoot version > 2.4 (as far as I remember). Looking at the pom file, this is 2.2.6. I'd recommend you to update SpringBoot version to 2.5 or above and test.
@nsundara
Copy link
Contributor Author

Hi Nirmala, in order for this to work with the newer values (...oracleucp...) you have to use a SpringBoot version > 2.4 (as far as I remember). Looking at the pom file, this is 2.2.6. I'd recommend you to update SpringBoot version to 2.5 or above and test.
@nsundara
Copy link
Contributor Author

I updated the POM file to 2.7.0 and verified it too. Can you approve?
@nsundara nsundara requested a review from psilberk May 25, 2022 20:59
@nsundara nsundara closed this Jun 2, 2022
@nsundara
Copy link
Contributor Author

nsundara commented Jun 2, 2022

The changes are merged as part of pull request #207
@gvenzl gvenzl deleted the spring-ucp branch January 31, 2023 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@nsundara @Kuassim @psilberk