feat: set custom_referrer if projects visited from org menu #104902
Conversation
github-actions
bot
added
the
Scope: Frontend
| try { | ||
| // pull the referrer from local storage. | ||
| if ( | ||
| localStorage.getItem('customReferrer') && |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
There was a problem hiding this comment.
Also, my gut is telling me that maybe we should move this below the query param block because it seems like we should treat the query param with priority since it's more explicit (i.e. visible to the user in the URL)
It's possible it could be confusing if somehow both of these were provided, and the more visible parameter was being ignored
There was a problem hiding this comment.
Pulled this into a const, also switched precedence.
![sentry[bot]](https://avatars.githubusercontent.com/u/1396951?s=60&v=4){kind=small}
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
bcoe
force-pushed
the
set-referrer-org-menu
branch
from
1e2f075 to
5b9b63c
Compare
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
We're curious to know what percentage of people visiting the Projects page are coming there from the org dropdown:
This adds
custom_referrertracking specifically to opening the project page, for the benefit of Amplitude.I added the ability to do so using
localStorage, so that we don't muck up the URL with the?referrer=fooquery parameters (we already litter these all over the place, and people have complained about cruft in URIs).