Skip to content

Update Vuex docs to remove reference to createLocalVue

Thomas Randolph requested to merge docs/local-vue into master

What does this MR do?

In #220482 we note that some docs are out of date.

This MR updates the docs for Vuex to remove the reference to createLocalVue (and the subsequent localVue variable).

See the review app for the changes as they would look when deployed.

It doesn't seem harmful to remove it from our "recommended" test setup, but it is actively confusing to leave it.

In addition to removing the unnecessary code, this MR also adds a note for people who might find createLocalVue and want to understand why it's not used any more. That note also links to the deprecation issue (linked above) to complete the loop.

Screenshots (strongly suggested)

Before After
Code image image
Note (No additional note) image

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Thomas Randolph

Merge request reports