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

fix for Grails 4 / Spring Security 5 based Spring-Security-Core plugin #397

Merged
merged 6 commits into from
May 10, 2019

Conversation

rvanderwerf
Copy link

Because of breaking changes this version can only work for Grails 4. (Like with Spring Security Core, UI, ACL, etc)

@rvanderwerf
Copy link
Author

We'll need to wait for the next Grails 4 milestone or RC release do be able to deploy this - it only works with a snapshot of current code, and the tests rely on sdkman which can't handle that. Once the next Grails 4 release milestone or candidate is released, we can just update the Grails version in this plugin and it should be good to go.

@rvanderwerf
Copy link
Author

also note datastore-rest-client 6.1.12 needs to be released, or forked to a 7.0.0 line for GORM before this can go as well.

rvanderwerf added 3 commits April 17, 2019 09:38
…n - fixes for integration tests and deps - using Grails 4.0.0.RC1 now
…n - fixes for integration tests and deps - fix build files for artifactory deploy issue
@patrikeev
Copy link

When are you going to release this? waiting to upgrade to grails 4, but I'm using this plugin...

@rvanderwerf
Copy link
Author

We are working on a way release datastore-rest-client 6.1.12 since it's on a snapshot at the moment. Hopefully this week!

@pmmakela
Copy link

pmmakela commented May 6, 2019

Any news on the release date?

@alvarosanchez alvarosanchez merged commit 68c3a6d into develop May 10, 2019
@alvarosanchez alvarosanchez deleted the grails4upgrade branch February 13, 2020 12:04
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.

None yet

4 participants