Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Master sync PR following v10.34.1 #20192

Merged
merged 8 commits into from
Jul 26, 2023
Merged

Master sync PR following v10.34.1 #20192

merged 8 commits into from
Jul 26, 2023

Conversation

danjm
Copy link
Contributor

@danjm danjm commented Jul 26, 2023

No description provided.

Gudahtt and others added 8 commits July 18, 2023 16:50
* Fix invalid state persistence error

We have been seeing Sentry errors showing that state persistence has
been failing for some users that have invalid `NetworkController`
state. This has been fixed by updating to
`@metamask/[email protected]`, which is more tolerant of
unexpected state properties.

* Update LavaMoat policies

---------

Co-authored-by: MetaMask Bot <[email protected]>
* Update sentry/cli to 2.19.4

* Ensure sentry files are loaded and referenced with a valid url

* Temp to eliminate errors in sentry (should be split into other PRs)
* Migration 89: ensure providerConfig in state has an id property

* Exit transformState function early if providerConfig already has an id

* Update migrations/index.js

* Code cleanup
@danjm danjm requested a review from a team as a code owner July 26, 2023 04:07
@github-actions
Copy link
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@codecov
Copy link

codecov bot commented Jul 26, 2023

Codecov Report

Merging #20192 (2c763a4) into develop (98279b6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop   #20192   +/-   ##
========================================
  Coverage    68.69%   68.69%           
========================================
  Files          986      986           
  Lines        37815    37815           
  Branches     10116    10116           
========================================
  Hits         25977    25977           
  Misses       11838    11838           

@metamaskbot
Copy link
Collaborator

Builds ready [2c763a4]
Page Load Metrics (1479 ± 43 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint108176125189
domContentLoaded1367176714799043
load1367176714799043
domInteractive1367176714799043
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@Gudahtt Gudahtt merged commit 6315ff8 into develop Jul 26, 2023
@Gudahtt Gudahtt deleted the master-sync branch July 26, 2023 11:43
@github-actions github-actions bot locked and limited conversation to collaborators Jul 26, 2023
@metamaskbot metamaskbot added the release-10.36.0 Issue or pull request that will be included in release 10.36.0 label Jul 26, 2023
@Gudahtt Gudahtt added release-11.1.0 Issue or pull request that will be included in release 11.1.0 and removed release-10.36.0 Issue or pull request that will be included in release 10.36.0 labels Sep 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release-11.1.0 Issue or pull request that will be included in release 11.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants