Skip to content

Remove vuex after vue-apollo implementation

Nicolò Maria Mezzopera requested to merge 276432-remove-vuex into master

What does this MR do?

This MR is the final step of the refactor from vuex to vue-apollo/graphql, we are removing the store and moving shared data to provide/inject

Plan

Description MR
Swap axios/vuex for vue apollo on list page !48602 (merged)
Swap axios/vuex for vue apollo on details page !49342 (merged)
Remove vuex and move what is left to provide inject 👈 you are here

Screenshots (strongly suggested)

No visual changes

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

Related to #276432 (closed)

Edited by Nicolò Maria Mezzopera

Merge request reports