-
Notifications
You must be signed in to change notification settings - Fork 61
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
Grails 3.3.x Support #161
Comments
Hey, What still needs to be done to upgrade the plugin to 3.3.x? Is there a road map, or any opportunities to submit PRs? |
I think the GORM Api has changed. Didn't have time to look at it. PR welcome. |
@robertoschwald The changes I made already support 3.3.x and we are using it today with 3.3.6 and GORM 6.1.10. I think we just need to officially release my Trait rewrite. |
@sudosandwich3 Take a look at the docs at https://robertoschwald.github.io/grails-audit-logging-plugin/snapshot/plugin.html. The current master branch is 3.0.0-SNAPSHOT and is a large rewrite of the API but works with Grails 3.3.x. The migration isn't too difficult and the docs should explain the main changes. I would also like to see the official 3.0.0 version released. |
Crap, looks like the documentation formatting has gotten messed up at some point along the way. I'll try to push a change to clean it up. |
@longwa Goodto know it works with 3.3.6 as well. I will bake a release then. |
@robertoschwald I pushed a change to fix the documentation formatting issue and remove the references to the Stamp support (I think Dennie pulled that stuff into a separate plugin). I'm not sure how you were publishing the docs but if you pull now and regenerate it should be good. |
Thanks for the change. Plugin is still supporting Stampable. Dennie has started PR #167 which is not finished, yet. I revert the doc files for Stampable refs. |
Need to migrate gradle build file for grailsPublish plugin, also. |
The text was updated successfully, but these errors were encountered: