Skip to content
Snippets Groups Projects

Update analytics dashboard list to match MVC design

Merged Jiaan Louw requested to merge 381322-product-analytics-dashboards-list into master
All threads resolved!

What does this MR do and why?

This MR updates the Product Analytics dashboard list to match the design as described in #381322 (closed).

This change is hidden behind a default off feature flag.

Screenshots or screen recordings

header After
image image

How to set up and validate locally

  1. You need to setup the product analytics devkit if you haven't already done so. https://gitlab.com/gitlab-org/analytics-section/product-analytics/devkit
  2. Enable the feature flag: echo "Feature.enable(:product_analytics_internal_preview)" | rails c.
  3. Visit the dashboard page of any project and validate that it renders: http://localhost:3000/gitlab-org/gitlab-shell/-/product_analytics/dashboards/

This will show an empty dashboard if you have no events in clickhouse. To generate events:

  1. You can use a browser extension like tapermonkey + this userscript to generate events.
    • Using the userscript above you need to change the host to your devkit's Jitsu URL and use your Jitsu's client key.

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

Edited by Jiaan Louw

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
  • Axel García approved this merge request

    approved this merge request

  • Axel García requested review from @rob.hunt and removed review request for @agarciatesares

    requested review from @rob.hunt and removed review request for @agarciatesares

  • :wave: @agarciatesares, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline will be started shortly.

    For more info, please refer to the following links:

  • Jiaan Louw added 1 commit

    added 1 commit

    Compare with previous version

  • Jiaan Louw added 1 commit

    added 1 commit

    Compare with previous version

  • Robert Hunt
  • Robert Hunt
  • Robert Hunt removed review request for @rob.hunt

    removed review request for @rob.hunt

  • Jiaan Louw added 1 commit

    added 1 commit

    • c1d3b1a8 - Apply maintainer suggestions

    Compare with previous version

  • Jiaan Louw requested review from @rob.hunt

    requested review from @rob.hunt

  • Robert Hunt resolved all threads

    resolved all threads

  • Robert Hunt approved this merge request

    approved this merge request

  • Robert Hunt enabled an automatic merge when the pipeline for 68e5f76e succeeds

    enabled an automatic merge when the pipeline for 68e5f76e succeeds

  • merged

  • Robert Hunt mentioned in commit 94d4fcea

    mentioned in commit 94d4fcea

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading