apps icon indicating copy to clipboard operation
apps copied to clipboard

feat: add share tracking URL query parameters and shorten the links

Open kopancek opened this issue 11 months ago • 1 comments

Changes

Adds userId and cid query params to all shared URLs. Minifies the URL before sharing. Modifies copyLink to send the same URL as well, in case copyLink feature flag is on.

Events

Did you introduce any new tracking events? Don't forget to update the Analytics Taxonomy sheet

Log the new/changed events below:

Type event_name value
Change/New event name extra: { ... }

Please make sure existing components are not breaking/affected by this PR

Manual Testing

On those affected packages:

  • [X] Have you done sanity checks in the webapp?
  • [X] Have you done sanity checks in the extension?
  • [ ] Does this not break anything in companion?

Did you test the modified components media queries?

  • [ ] MobileL (420px)
  • [ ] Tablet (656px)
  • [X] Laptop (1020px)

Did you test on actual mobile devices?

  • [ ] iOS (Chrome and Safari)
  • [ ] Android

MI-188 #done

kopancek avatar Mar 07 '24 14:03 kopancek

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
daily-webapp ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 8, 2024 0:43am
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
storybook ⬜️ Ignored (Inspect) Mar 8, 2024 0:43am

vercel[bot] avatar Mar 07 '24 14:03 vercel[bot]