MetaMask / MetaMask/core

Introduce concept of `PermissionGroups` to the `PermissionController`

Open
#1,459 4 comments 0 reactions 1 assignee Claimed by @hmalik88 View on GitHub
team-wallet-framework wf-permission-controller-improvements
Dominant language
TypeScript
Stars
413
Forks
308
Avg merge
1d 4h
Merged PRs (30d)
253

Description

As it stands, we currently have a 1 to 1 mapping from permission to rpc method. With the addition of multiple rpc methods that fall around the same action or permission, it is necessary for the `PermissionController` to be able to process a permission request for a `parent` permission and subsequently install the `child` permissions. This greatly reduces the visual overload you get from seeing multiple related permissions in the connect screen.

The first permission to have a group would be `snap_manageState` and then eventually the forthcoming custom ui rpc methods.

Spec in Notion: https://www.notion.so/Permission-Groups-7a4047730e8745c2ac17d7714f791dcc

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.