Skip to content

track installs on distro page#5621

Merged
ilayda-cp merged 2 commits intomainfrom
WD-34921-track-distro-install
Mar 16, 2026
Merged

track installs on distro page#5621
ilayda-cp merged 2 commits intomainfrom
WD-34921-track-distro-install

Conversation

@ilayda-cp
Copy link
Copy Markdown
Contributor

@ilayda-cp ilayda-cp commented Mar 13, 2026

Done

  • Added copy buttons to commands in the distro page
  • track install command copy button

How to QA

  • Go to a distro install page ie https://snapcraft-io-5621.demos.haus/install/vault/arch
  • Verify copy buttons appear on all command snippets
  • Click a copy button
  • confirm text is copied to clipboard and icon changes to checkmark
  • open browser Network tab
  • click on the copy button on the install command (snap install ....)
  • wait for a couple of secs
  • verify click event fires with target snap_distro_install_copy_install_command
  • verify a page_view event fires with target snap_distro_install_page

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-34921

Screenshots

UX Approval

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

@webteam-app
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@abbiesims abbiesims left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

@ilayda-cp ilayda-cp merged commit 3b70bc8 into main Mar 16, 2026
14 checks passed
@ilayda-cp ilayda-cp deleted the WD-34921-track-distro-install branch March 16, 2026 06:10
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.

3 participants