Skip to content

Add Snowplow Events for Data Studio #66383

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add Snowplow Events for Data Studio #66383

wants to merge 2 commits into from
+402 −85

Conversation

@andsalves
Copy link
Contributor

@andsalves andsalves commented Nov 28, 2025

Closes https://linear.app/metabase/issue/UXW-2287/add-snowplow-events-for-data-studio

Description

Adding missing snowplow events to Data Studio pages.

How to verify

  • Run Metabase with snowplow enabled
  • Check out the list of expected events in Data Studio and check that they are triggered accordingly:
    • data_studio_opened
    • data_studio_library_created
    • data_studio_library_model_created
    • data_studio_library_metric_created
    • data_studio_table_published
    • data_studio_dependency_graph_opened
    • data_studio_glossary_term_created
    • data_studio_glossary_term_updated
    • data_studio_glossary_term_deleted

Checklist

  • Tests have been added/updated to cover changes in this PR

@andsalves andsalves added the no-backport Do not backport this PR to any branch label Nov 28, 2025
@metabase-bot metabase-bot bot added the .Team/UXWest UX West - use for bugs and feature requests label Nov 28, 2025
@andsalves andsalves requested review from a team, iethree and npfitz November 28, 2025 03:18
@github-actions
Copy link
Contributor

github-actions bot commented Nov 28, 2025

e2e tests failed on 48f7bc9ee75ec967e5afbfde8c518e184c005b03-1

e2e test run

File Test Name
embedding-dashboard.cy.spec.js scenarios > embedding > dashboard parameters > should allow searching dashboard parameters in preview embed modal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@npfitz npfitz Awaiting requested review from npfitz npfitz was automatically assigned from metabase/core-frontend-ux-west

@iethree iethree Awaiting requested review from iethree iethree was automatically assigned from metabase/core-frontend-ux-west

At least 1 approving review is required to merge this pull request.

Assignees

@andsalves andsalves

Labels

no-backport Do not backport this PR to any branch .Team/UXWest UX West - use for bugs and feature requests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@andsalves