Add metrics events for Wyre and CoinSwitch #8677
Conversation
| onButtonClick: () => { | ||
| this.context.metricsEvent({ | ||
| eventOpts: { | ||
| category: 'Onboarding', |
There was a problem hiding this comment.
Does the Onboarding category make sense here? 🤔
There was a problem hiding this comment.
It is primarily a way to onboard people
There was a problem hiding this comment.
Not really? It's a way to buy Eth. Existing users need to buy Eth sometimes too. It's not even presented during onboarding.
There was a problem hiding this comment.
Updated to use "Accounts"
There was a problem hiding this comment.
Let me know if you have a different suggestion
There was a problem hiding this comment.
Maybe "On-Ramp"? Or "Integrations", or "Third-Party", or "Exchange"? 🤷♂️
A new category would just have this one action type in it, but that's not unprecedented. We'll likely have more related metrics soon as well.
I guess "Accounts" is fine too.
Closes #7699
This PR adds metrics events for the ETH purchase buttons.