Fix escaping dismissal descriptions on MR widget
What does this MR do and why?
Related #460639 (closed)
In French, the dismissal descriptions which get injected on the MR page escapes the string and causes a SyntaxError. This is fixed by using backticks `.
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Before | After |
---|---|
How to set up and validate locally
Prerequisites
- You need an EE license
- You need to have runners enabled (See $2408961 for setting up a runner)
- Import https://gitlab.com/gitlab-org/govern/demos/sandbox/minac/test-remediations
- Do not run a pipeline yet on master!
- In the imported test-remediations project
- go to
reports/sast.json
and remove both entries invulnerabilities
, leaving effectively an empty array - commit and push this to master
- now, in
reports/sast.json
, add the removed vulnerabilities back again, but commit and push to a new branch, and create a merge request.
- go to
- Go to http://gdk.test:3000/-/profile/preferences and set language to French
- Go to the MR you just created and validate that you do see the License Compliance widget and Security Report widget
Edited by Lorenz van Herwaarden