Skip to content

Replace v-html with gl-sprintf component in locked_warning.vue (#241850)

Stanislav Lashmanov requested to merge cyberap/gitlab:lashmanov/241850-replace-v-html-with-sprintf into master

What does this MR do and why?

Replaces v-html in app/assets/javascripts/issue_show/components/locked_warning.vue with gl-sprintf component which does exactly the same thing that sprintf helper function does, but in a component-friendly way.

Closes #241850 (closed).

Screenshots or screen recordings

Before After
image image

How to set up and validate locally

  1. Open an issue you can edit in the first tab and hit 'Edit' button
  2. Open the same page in the second tab and also hit 'Edit' button
  3. Save the issue in the second tab

The warning message should appear in the first tab with the clickable link to the issue you're viewing

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Stanislav Lashmanov

Merge request reports