Skip to content

integrate metrics & add metrics for snap detail page#5604

Merged
ilayda-cp merged 14 commits intomainfrom
WD-32982-prod-metrics-integration
Mar 3, 2026
Merged

integrate metrics & add metrics for snap detail page#5604
ilayda-cp merged 14 commits intomainfrom
WD-32982-prod-metrics-integration

Conversation

@ilayda-cp
Copy link
Copy Markdown
Contributor

@ilayda-cp ilayda-cp commented Feb 26, 2026

Done

Integrates the @canonical/analytics-events package into snapcraft.io to track user interactions on snap detail pages.

How to QA

  • go to https://snapcraft-io-5604.demos.haus/artikulate
  • open to dev console
  • go to network tab
  • scroll to the Where people are using <snap_name> section
  • hover, click etc
  • wait for 5-10 secs verify "events" request is sent with correct values

Testing

  • This PR has tests
  • No testing required (explain why):

Security

  • Security considerations for review (list them):
    • Examples:
    • Access control: users can only access their own data
    • Input: user input is validated and sanitised
    • Sensitive data: secret or private data is not exposed in any way
    • ...
  • This PR has no security considerations (explain why):

Issue / Card

Fixes https://warthogs.atlassian.net/browse/WD-32982

UX Approval

  • This PR does not require UX approval
  • This PR does require UX approval (add context):

@webteam-app
Copy link
Copy Markdown

Comment thread .github/workflows/pr.yml Outdated
Comment thread konf/staging-api.snapcraft.io.yaml Outdated
Comment thread konf/site.yaml Outdated
Copy link
Copy Markdown
Contributor

@steverydz steverydz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Copy link
Copy Markdown
Contributor

@edisile edisile left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm! not sure what caused the big change to yarn.lock but you might want to remove that and rebase on main when merging

@ilayda-cp
Copy link
Copy Markdown
Contributor Author

@edisile i added a package "@canonical/analytics-events"

@ilayda-cp ilayda-cp merged commit 9ad88db into main Mar 3, 2026
14 checks passed
@ilayda-cp ilayda-cp deleted the WD-32982-prod-metrics-integration branch March 3, 2026 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants