-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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: fix network verification chainID and name verification for popular network #8953
Conversation
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. |
1cd403a
to
da73bbc
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8953 +/- ##
=======================================
Coverage 43.67% 43.67%
=======================================
Files 1271 1271
Lines 30921 30921
Branches 3092 3092
=======================================
Hits 13506 13506
Misses 16636 16636
Partials 779 779 ☔ View full report in Codecov by Sentry. |
3542db0
to
90da418
Compare
90da418
to
3a46b1d
Compare
0a54e66
to
c8c98ce
Compare
c8c98ce
to
b534824
Compare
Bitrise❌❌❌ Commit hash: b534824 Note
|
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
Bitrise✅✅✅ Commit hash: 143e0c2 Note
|
Quality Gate passedIssues Measures |
Description
the chainId must be displayed in decimal instead of hexadecimal in the network verification modal.
the name warning must not be displayed in the case of a popular network.
Related issues
Fixes: #8950 , #8951
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist