Copilot commented on code in PR #16057:
URL: https://github.com/apache/grails-core/pull/16057#discussion_r3661452944
##########
grails-doc/src/en/guide/security/securityPlugins/springSecurity/ui/customization.adoc:
##########
@@ -177,7 +177,7 @@ The layouts use `grails-app/assets/javascripts/jquery.js`,
which contains this:
//=require
jquery/jquery-${grails.plugin.springsecurity.ui.Constants.JQUERY_VERSION}.js
----
-This resolves to `grails-app/assets/javascripts/jquery/jquery-2.1.4.js`, and
to use your own version, either use the same approach in a file called
`jquery.js` or rename your file to `jquery.js`.
+This resolves to `grails-app/assets/javascripts/jquery/jquery-4.0.0.js`, and
to use your own version, either use the same approach in a file called
`jquery.js` or rename your file to `jquery.js`.
Review Comment:
This section describes plugin-provided
`grails-app/assets/javascripts/jquery.js` / `jquery-ui.js` files that use
`${grails.plugin.springsecurity.ui.Constants.*}` to resolve to versioned
assets. In the current plugin, the layout includes `spring-security-ui.js`
(which directly `//= require`s webjars paths), and there is no `jquery.js` /
`jquery-ui.js` asset in the plugin. The documentation should be updated to
reflect the current asset layout and override mechanism, otherwise readers will
follow non-existent paths.
This issue also appears on line 200 of the same file.
--
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]
