Skip to content
Snippets Groups Projects

Value Stream analytics: Move tasks by type request out of Vuex action

Merged Alex Pennells requested to merge 477817-load-tasks-in-component into master
All threads resolved!

What does this MR do and why?

This is part of a continued effort to remove the typeOfWork Vuex module used by the Value stream analytics app. This MR begins to migrate the Vuex actions into the TypeOfWorkCharts component.

Summary:

  • Replaces fetchTasksByTypeData Vuex action with fetchTasksByType component method
  • Replaces data Vuex state attribute with tasksByType component data attribute

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

No visual changes

Screenshot_2024-08-13_at_3.38.02_PM

How to set up and validate locally

  1. Ensure you are using GitLab Ultimate
  2. Create a new Project
  3. Generate mock data: FILTER=customizable_cycle_analytics SEED_CUSTOMIZABLE_CYCLE_ANALYTICS=1 VSA_SEED_PROJECT_ID=${PROJECT_ID} bundle exec rake db:seed_fu
  4. Navigate to VSA for the owning group of the project created: Analyze > Value stream analytics
  5. Validate that the Tasks by type chart (bottom of the Overview tab) works with different labels/Merge requests

Related to #477817 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Ezekiel Kigbo
  • mentioned in issue #492101 (closed)

  • Ezekiel Kigbo resolved all threads

    resolved all threads

  • Ezekiel Kigbo approved this merge request

    approved this merge request

  • Great work @apennells thanks for initiating and pushing forward with this refactor and thanks for the good discussions, i've opened a follow-up issue but can go ahead and merge this for now.

  • Ezekiel Kigbo enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • Ezekiel Kigbo added this merge request to the merge train at position 2

    added this merge request to the merge train at position 2

  • Ezekiel Kigbo mentioned in commit cbc2e182

    mentioned in commit cbc2e182

  • added workflowstaging label and removed workflowcanary label

  • Alex Pennells mentioned in merge request !167487 (merged)

    mentioned in merge request !167487 (merged)

  • Alex Pennells mentioned in merge request !174195 (merged)

    mentioned in merge request !174195 (merged)

  • Please register or sign in to reply
    Loading