Skip to content

Added dropdown with predefined date ranges to Value Stream Analytics

Rudy Crespo requested to merge 408656-vsa-predefined-date-range into master

What does this MR do and why?

Adds a dropdown with predefined date ranges to Value Stream Analytics' filters.

Screenshots or screen recordings

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

View Before After After (custom option selected)
Group level Screenshot 2023-09-18 at 3.00.58 PM.png Screenshot 2023-10-04 at 11.32.45 AM.png Screenshot 2023-10-04 at 11.33.00 AM.png
Project level Screenshot 2023-09-18 at 3.00.39 PM.png Screenshot 2023-10-04 at 11.33.13 AM.png Screenshot 2023-10-04 at 11.33.28 AM.png
FOSS (project level only) Screenshot 2023-09-18 at 3.09.44 PM.png Screenshot 2023-10-04 at 11.35.38 AM.png Screenshot 2023-10-04 at 11.35.19 AM.png

How to set up and validate locally

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

  1. Please make sure that you're on the GitLab Ultimate plan.
  2. Feature.enable(:vsa_predefined_date_ranges)
  3. Visit any group/project that has VSA data or follow these steps:
    1. run SEED_CYCLE_ANALYTICS=true SEED_VSA=true FILTER=cycle_analytics rake db:seed_fu
    2. Visit the group value stream. The group URL is present in the output of the command above.
    3. Open rails console and run Analytics::CycleAnalytics::ReaggregationWorker.new.perform
    4. Refresh the group value stream and you should see data
  4. Verify that the dropdown works correctly.
    • Predefined dates ranges return correct start and end dates, as well as number of days selected
    • Selecting a predefined date range hides date range picker
    • Selecting Custom option displays date range picker
  5. Repeat Step 4 for CE as well (project level only)

MR acceptance checklist

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

Related to #408656 (closed)

Edited by Rudy Crespo

Merge request reports