Skip to content

Fix external sourcemaps reference causing error in browser console#8367

Merged
kumavis merged 1 commit intobuild-sys-2from
build-sys-2-sourcemaps
Jan 18, 2021
Merged

Fix external sourcemaps reference causing error in browser console#8367
kumavis merged 1 commit intobuild-sys-2from
build-sys-2-sourcemaps

Conversation

@kumavis
Copy link
Copy Markdown
Member

@kumavis kumavis commented Apr 20, 2020

Blocked by #8170
Fixes #8167

current PR target is build-sys-2 to correctly show diffs for this

need to see if this causes sentry to not be able to parse callstacks. having trouble testing this. may have to merge to test in prod and revert if its a problem

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [0ed6409]
Page Load Metrics (677 ± 50 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeNotificationfirstPaint35143482311
domContentLoaded48487067510350
load48687167710450
domInteractive48487067410350

@kumavis
Copy link
Copy Markdown
Member Author

kumavis commented Jan 18, 2021

merging this into build-sys-2, which is highly conflicted and wont land as is

@kumavis kumavis merged commit 5e47cf9 into build-sys-2 Jan 18, 2021
@kumavis kumavis deleted the build-sys-2-sourcemaps branch January 18, 2021 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants