Skip to content

Add queryOverrides to analytics dashboard GraphQL query

What does this MR do and why?

This MR adds fetches the queryOverrides when querying the Product Analytics Dashboard when the product_analytics_snowplow_support feature flag is enabled. Previously there was an oversight and this field was not included in the GraphQL response nor query. The response has been updated with Add queryOverrides field to product analytics d... (!124065 - merged).

Note: This change is dependant on Add queryOverrides field to product analytics d... (!124065 - merged) being deployed to all environments. [DONE]

Note: This change is experimental and hidden behind a default off feature flag.

Screenshots or screen recordings

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

No visual changes.

Before After
image image

How to set up and validate locally

  1. Make sure you are on GitLab Ultimate
  2. Follow these instructions to set up Product Analytics in your GDK.
    • NB enable the product_analytics_snowplow_support feature flag.
  3. Create a new project with an empty repository.
  4. Enable custom dashboards at Project > Settings > Analytics and pick the empty project.
  5. Create a custom dashboard by following these instructions with the contents below:
  6. View the Something dashboard at Project > Settings > Analytics and use the browser's network inspector to verify that the query's limit is 200.
  7. Checkout master branch and repeat the previous steps. Except verify that the query's limit is 100 during the last step.
---
version: 1
title: Something
description: All insights on user activity
panels:
  - id: 1
    visualization: page_views_over_time
    title: Page views over time
    gridAttributes:
      yPos: 1
      xPos: 0
      width: 6
      height: 5
    options: {}
    queryOverrides:
      limit: 200

image

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

Edited by Jiaan Louw

Merge request reports