Skip to content

Remove panel via vuex in dashboard

Miguel Rincon requested to merge fix-remove-metric-in-dashboard into master

What does this MR do?

Background Moving forward our dashboard will be saved back to the backend, so we need the frontend to keep an update vuex store if the user makes changes.

Issue Currently the use can remove a panel from the dashboard (when rearrangePanelsAvailable is true) however, the change is done by removing the object directly from the object in the dashboard and skipping a commit to the store.

This fix ensures this is done using a Vuex store action instead.

You can enable rearrangePanelsAvailable in the dashboard to see the draggable/removable panels in action.

Screenshots

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 #27835 (closed)

Edited by 🤖 GitLab Bot 🤖

Merge request reports