-
Notifications
You must be signed in to change notification settings - Fork 9
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
Chore: Admin - Add jQuery to agency base template #2329
Conversation
<script nonce="{{ request.csp_nonce }}" | ||
src="https://cdn.jsdelivr.net/npm/[email protected]/dist/jquery.min.js" | ||
integrity="sha256-o88AwQnZB+VDvE9tvIXrMQaPlFFSUTR+nldQm1LuPXQ=" | ||
crossorigin="anonymous"></script> | ||
{% endblock extrastyle %} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@angela-tran Should I also add this line from /calitp/app/benefits/core/templates/core/base.html
to pull in the recaptcha script, in this PR?
{% if request.recaptcha %}
<script nonce="{{ request.csp_nonce }}" src="{{ request.recaptcha.script_api }}"></script>
{% endif %}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would have preferred to be able to add this to the very bottom of the <body>
tag, but there isn't an easy place to override and add this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FWIW, I think we can ignore reCAPTCHA for this effort -- the user must have access to the Admin, and I think we can assume they are legit if they can get in and initiate an in-person flow.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
closes #2328
This PR adds jQuery to the
agency-base
template, so all Admin pages that use this template will have access to jQuery. jQuery is used inform.html
- the main form base template that the app uses for all forms.