-
-
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
feat(2739): permission summary view gets the ability to disconnect all #11239
feat(2739): permission summary view gets the ability to disconnect all #11239
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. |
…r adding the disconnect all button
…Edit button link that was there before
…lete and stable then we will remove this exclusion and write unit tests
…this is now fixed
…lete and stable then we will remove this exclusion and write unit tests
…ing bool prop, just need to negate the other instead
Bitrise✅✅✅ Commit hash: a2e54bf Note
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #11239 +/- ##
==========================================
+ Coverage 55.24% 55.75% +0.50%
==========================================
Files 1558 1580 +22
Lines 37153 37646 +493
Branches 4406 4507 +101
==========================================
+ Hits 20527 20988 +461
- Misses 16155 16168 +13
- Partials 471 490 +19 ☔ View full report in Codecov by Sentry. |
Quality Gate passedIssues Measures |
Description
The permission summary view gets the ability to diconnect all
Related issues
Contributes to solve issue: https://github.com/MetaMask/MetaMask-planning/issues/2796
Manual testing steps
I have added screenshot below to see how it looks.
Screenshots/Recordings
Pre-merge author checklist
Pre-merge reviewer checklist