-
-
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(2794): ticket account permission selection #11582
feat(2794): ticket account permission selection #11582
Conversation
…complete, we plan to start writing tests next week
…now fixed And was clearing the selected accounts of the edit all accounts flow, and removing accounts was buggy if all permitted were removed before adding a new permitted account, all this is fixed in this commit
…hen this component is used in the context of connecting more accounts
…scription telling the user to select an account when there is none to select
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. |
… has occured, this feature is only for when feature flag is on
Bitrise✅✅✅ Commit hash: 369dcd3 Note
|
…can do, for example he can add and remove in the same click of the update button
Quality Gate passedIssues Measures |
ensByAccountAddress={ensByAccountAddress} | ||
selectedAddresses={selectedAddresses} | ||
selectedAddresses={effectiveSelectedAddresses} |
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.
Refs are probably not needed here. This value just needs to be selectedAddresses
with the default permitted accounts and set at the same time as navigation if needed.
closing this PR since it is now replaced by #11748 |
Description
This PR adds the ability to edit account permissions for in app browser dapps.
Related issues
Contributes to solve issue: https:/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