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

Resolved: org.grails.web.converters.configuration.ConvertersConfigura… #3

Merged
merged 2 commits into from
Nov 22, 2017

Conversation

AFSDevGuy
Copy link
Contributor

…tionInitializer does not respect grails.converters.include.class settings for the "deep" configuration #2

…tionInitializer does not respect grails.converters.include.class settings for the "deep" configuration grails#2
@jameskleeh
Copy link
Contributor

This PR needs tests

@AFSDevGuy
Copy link
Contributor Author

Good point. The current DomainClassMarshallerSpec.groovy test could likely be adapted to both demonstrate the current issue as well as the fix. I'll take a look at making those changes as well. Thanks!

…logic to XML marshaller to emit class name as an attribute.
@graemerocher graemerocher merged commit e6904be into grails:master Nov 22, 2017
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.

3 participants