Skip to content

Remove Vue set for vue 3 migration

Artur Fedorov requested to merge 452467-vue3-migration-vulnerability_list into master

What does this MR do and why?

Describe in detail what your merge request does and why.

This MR partially resolves #452467 (closed)

What was cleaned up from epic:

  1. Remove Vue.set

Vue set is no longer supported in VUE 3

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

NO VISUAL CHANGES

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Checkout branch

  2. Run test locally

  3. Or just check in pipeline

  4. Run it also with VUE_VERSION=3 yarn jest --watch 

  5. Go to Group/Projects

  6. Secure -> Vulnerability Report

  7. Submit 1-2 Test vulnerabilities

  8. Go back to Vulnerability report

  9. Toggle items in table

MR acceptance checklist

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

Merge request reports