-
Notifications
You must be signed in to change notification settings - Fork 5k
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
fix: Remove obsolete NetworkController state #26302
Conversation
Blocked by #26298 |
6110994
to
b2ea75d
Compare
Migration 120.2 has been updated to remove obsolete NetworkController state as well. Fixes #26297
b2ea75d
to
c17e849
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #26302 +/- ##
===========================================
+ Coverage 70.02% 70.03% +0.01%
===========================================
Files 1411 1411
Lines 49995 50008 +13
Branches 13805 13807 +2
===========================================
+ Hits 35006 35019 +13
Misses 14989 14989 ☔ View full report in Codecov by Sentry. |
Builds ready [c17e849]
Page Load Metrics (311 ± 267 ms)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
cc38f47
to
c17e849
Compare
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Migration 120.2 has been updated to remove obsolete NetworkController state as well.
Related issues
Fixes #26297
Manual testing steps
N/A
Screenshots/Recordings
N/A
Pre-merge author checklist
Pre-merge reviewer checklist