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.

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

Fixing configuration loading with Micronaut (issue #11767) #11828

Merged
merged 5 commits into from
Sep 24, 2023
Merged

Fixing configuration loading with Micronaut (issue #11767) #11828

merged 5 commits into from
Sep 24, 2023

Conversation

davidkron
Copy link
Contributor

No description provided.

@davidkron
Copy link
Contributor Author

I get the following error from CI:

Run scacap/action-surefire-report@v1
Going to parse results form **/build/test-results/test/TEST-*.xml
Result: 1716 tests run, 23 skipped, 0 failed.
Posting status 'completed' with conclusion 'success' to https://github.com/grails/grails-core/pull/11828 (sha: 1c61fb2a855082fd54f408f4df66df4ee2ed4856)
Error: Resource not accessible by integration

Can someone support me on what the problem is?

@puneetbehl puneetbehl linked an issue Sep 17, 2021 that may be closed by this pull request
Grails 5.0.0.M2: property inside plugin has no effect on environment configuration (extension to #11519) #11767
Closed
4 tasks
@CLAassistant
Copy link

CLAassistant commented Nov 3, 2021 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@puneetbehl
Copy link
Contributor

@davidkron Thank you for your contribution, I am sorry for the delay. I will take a look in this week itself.

@puneetbehl puneetbehl changed the base branch from master to 5.2.x March 21, 2022 06:12
@jchharris
Copy link
Contributor

@puneetbehl Is there a plan to incorporate this into a release soon?

@matrei
Copy link
Contributor

matrei commented Aug 30, 2023

@puneetbehl I also ran into this issue today. Is there something holding this PR back?

@puneetbehl puneetbehl changed the base branch from 5.2.x to 5.3.x September 24, 2023 18:55
@puneetbehl puneetbehl merged commit 567a615 into grails:5.3.x Sep 24, 2023
@matrei
Copy link
Contributor

matrei commented Nov 6, 2023 •
edited
Loading

@puneetbehl The changes in this PR are not applied to branches 6.0.x and 6.1.x

matrei added a commit to matrei/pingcrm-grails that referenced this pull request Jan 9, 2024
@matrei
fix: Remove ssr.url from config
2980760 
As grails/grails-core#11828 is now merged, the default config value from the Inertia plugin is now picked up.
@matrei
Copy link
Contributor

matrei commented Feb 12, 2024

I am still having issues with #11767.
Example application showing the problem: https://github.com/matrei/mnPluginPropertiesBug

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Grails 5.0.0.M2: property inside plugin has no effect on environment configuration (extension to #11519)
5 participants
@davidkron @CLAassistant @puneetbehl @jchharris @matrei