Skip to content

Track a new schema event when adding a token#9810

Merged
brad-decker merged 6 commits intodevelopfrom
add-token-event
Nov 17, 2020
Merged

Track a new schema event when adding a token#9810
brad-decker merged 6 commits intodevelopfrom
add-token-event

Conversation

@brad-decker
Copy link
Copy Markdown
Contributor

Explanation:
The new schema has an event for 'Token Added', the intent of which is to surface popular tokens that are being added in spikes. This would help to inform support of the potential influx of issues as well as swaps for adding support for new token pairs.

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [622ad0d]
Page Load Metrics (349 ± 52 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint309639147
domContentLoaded24166734810752
load24366834910752
domInteractive24166734810752

@brad-decker brad-decker marked this pull request as ready for review November 5, 2020 21:41
@brad-decker brad-decker requested a review from a team as a code owner November 5, 2020 21:41
@brad-decker brad-decker requested a review from Gudahtt November 5, 2020 21:41
@darkwing
Copy link
Copy Markdown
Contributor

darkwing commented Nov 6, 2020

Not a blocker but do we want a trackEvents helper?

@brad-decker
Copy link
Copy Markdown
Contributor Author

@darkwing for tracking multiple events like this? See #9807 -- love to get your thoughts on the approach there!

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [8e678cf]
Page Load Metrics (397 ± 66 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint3010741199
domContentLoaded24678339513766
load24878539713766
domInteractive24678339513766

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [6e8a36e]
Page Load Metrics (421 ± 59 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint29119502211
domContentLoaded27273241912359
load27473342112359
domInteractive27273241812359

Copy link
Copy Markdown
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Found one issue, but everything looks good aside from that!

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [a0fe3e9]
Page Load Metrics (399 ± 65 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint29713794
domContentLoaded24567239713565
load24767439913665
domInteractive24567239713565

Copy link
Copy Markdown
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@brad-decker brad-decker merged commit daf783a into develop Nov 17, 2020
@brad-decker brad-decker deleted the add-token-event branch November 17, 2020 17:39
@github-actions github-actions bot locked and limited conversation to collaborators Nov 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants