Fix duplicated events tracking

gianfra-t commented 2 weeks ago

Closes #132.

This PR adds three simple changes:

Adds to the UNIQUE_EVENT_TYPES list all events that we are interested in tracking once per off-ramp flow. This should now prevent the duplicated transaction_success. I would prefer to use this method to keep idempotency instead of investigating why trackEvent gets called twice.
Adds a resetUniqueEvents function when the user finishes the off-ramp, that will allow to register again if the user does another one without refreshing.
Add a guard to check for wallet connect/disconnect events only if the user has clicked the wallet button. This allows to be resistant to changes in state upon refresh of the site page.

netlify[bot] commented 2 weeks ago

Name	Link
Latest commit	71a0c3fcf7a6b89a4edd3b096daaf50fff6c874b
Latest deploy log	https://app.netlify.com/sites/pendulum-pay/deploys/66e2af9d56d0a100089466b3
Deploy Preview	https://deploy-preview-150--pendulum-pay.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

gianfra-t commented 2 weeks ago

@pendulum-chain/devs tagging for review.

pendulum-chain / vortex