codeconsole commented on code in PR #15976:
URL: https://github.com/apache/grails-core/pull/15976#discussion_r3565488512
##########
grails-core/src/main/resources/META-INF/additional-spring-configuration-metadata.json:
##########
@@ -28,6 +28,12 @@
"description": "A list of versions to exclude from the banner.",
"type": "java.util.List<grails.boot.GrailsBanner.DefaultVersionOption>"
},
+ {
+ "name": "grails.converters.encoding",
Review Comment:
Fixed in 0a45f47908 — dropped the second declaration from grails-core and
broadened the existing grails-web-core entry in place to cover the respond()
renderer content types (which also resolves the banner-group placement nit).
One correction on the doc pointer: `ref/Configuration/Application
Properties.adoc` doesn't exist on 8.0.x and `grails.converters.encoding` wasn't
documented anywhere in grails-doc, so instead of editing stale wording I added
the setting to the runtime settings list in
`guide/conf/config/builtInOptions.adoc` with the broadened description.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
