1
0
mirror of https://github.com/kremalicious/metamask-extension.git synced 2024-12-23 09:52:26 +01:00

Adding gasEstimateType to 'Changed Gas Button' metrics event (#11352)

Adding 'Changed Gas Button' metrics event
This commit is contained in:
ryanml 2021-06-21 12:02:51 -07:00
parent cbb0e4d45c
commit fed9c86abb
2 changed files with 12 additions and 15 deletions

View File

@ -32,11 +32,11 @@ export default class SendGasRow extends Component {
static contextTypes = {
t: PropTypes.func,
metricsEvent: PropTypes.func,
trackEvent: PropTypes.func,
};
renderAdvancedOptionsButton() {
const { metricsEvent } = this.context;
const { trackEvent } = this.context;
const {
showCustomizeGasModal,
isMainnet,
@ -54,12 +54,9 @@ export default class SendGasRow extends Component {
<div
className="advanced-gas-options-btn"
onClick={() => {
metricsEvent({
eventOpts: {
trackEvent({
category: 'Transactions',
action: 'Edit Screen',
name: 'Clicked "Advanced Options"',
},
event: 'Clicked "Advanced Options"',
});
showCustomizeGasModal();
}}
@ -105,7 +102,7 @@ export default class SendGasRow extends Component {
isEthGasPrice,
noGasPrice,
} = this.props;
const { metricsEvent } = this.context;
const { trackEvent } = this.context;
const gasPriceFetchFailure = isEthGasPrice || noGasPrice;
const gasPriceButtonGroup = (
@ -115,11 +112,11 @@ export default class SendGasRow extends Component {
showCheck={false}
{...gasPriceButtonGroupProps}
handleGasPriceSelection={async (opts) => {
metricsEvent({
eventOpts: {
trackEvent({
category: 'Transactions',
action: 'Edit Screen',
name: 'Changed Gas Button',
event: 'User Clicked Gas Estimate Button',
properties: {
gasEstimateType: opts.gasEstimateType.toLowerCase(),
},
});
await gasPriceButtonGroupProps.handleGasPriceSelection(opts);

View File

@ -32,7 +32,7 @@ describe('SendGasRow Component', () => {
anotherGasPriceButtonGroupProp: 'bar',
}}
/>,
{ context: { t: (str) => `${str}_t`, metricsEvent: () => ({}) } },
{ context: { t: (str) => `${str}_t`, trackEvent: () => ({}) } },
);
wrapper.setProps({ isMainnet: true });
});