retire analytics events #1267
retire analytics events #1267
Conversation
sparrowDom
left a comment
sparrowDom
left a comment
There was a problem hiding this comment.
Awesome job with the cleanup 👏
Curious why are we cleaning up some of the analytics tracking events? I haven't commented under each one that I though still might stay in the codebase
| const metadata = swapMetadata() | ||
|
|
||
| try { | ||
| analytics.track('Before Swap Transaction', { |
There was a problem hiding this comment.
curious why are we removing this one?
| value = value > 50 ? 50 : value | ||
| value = truncateDecimals(value, 2) | ||
| if (value !== priceToleranceValue) { | ||
| analytics.track('On price tolerance change', { |
|
For the analytics tracking event I'm going off @jonathansnow's recommendations here https://docs.google.com/spreadsheets/d/1iWfCSBpxxAFu8yObAVKZCnlfk-d903RpgGCggT2Ex1A/edit#gid=0. I think for |
|
Yes, what Roland mentioned above is true. We're migrating everything to GTM/GA4 and moving away from third-party libraries. UA EOL is on July 1st. We've removed items we're not concerned about right now while noting them to be added back in the future possibly. |
remove some analytics event tracking code + clean up some old marketing pages content