Skip to content

Fix logWeb3ShimUsage metrics#10286

Merged
rekmarks merged 1 commit intodevelopfrom
fix-logWeb3ShimUsage
Jan 25, 2021
Merged

Fix logWeb3ShimUsage metrics#10286
rekmarks merged 1 commit intodevelopfrom
fix-logWeb3ShimUsage

Conversation

@rekmarks
Copy link
Copy Markdown
Member

The sendMetrics call in the metamask_logWeb3ShimUsage handler was... not correctly formatted. It should now be fixed. I couldn't find any usage of the referrer property, but according to the following type definition, it should be fine now: https://github.com/MetaMask/metamask-extension/blob/develop/shared/constants/metametrics.js/#L17-L22

@rekmarks rekmarks requested a review from a team as a code owner January 25, 2021 21:42
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [1befce7]
Page Load Metrics (515 ± 37 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint41735394
domContentLoaded3246185137837
load3266205157837
domInteractive3246185137837

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, great catch!

@rekmarks rekmarks merged commit 7ebab83 into develop Jan 25, 2021
@rekmarks rekmarks deleted the fix-logWeb3ShimUsage branch January 25, 2021 22:20
@github-actions github-actions bot locked and limited conversation to collaborators Jan 25, 2021
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.

3 participants