core icon indicating copy to clipboard operation
core copied to clipboard

Introduce concept of `PermissionGroups` to the `PermissionController`

Open hmalik88 opened this issue 1 year ago • 4 comments

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

hmalik88 avatar Jun 28 '23 11:06 hmalik88

From sprint planning: Pending review of the doc Hassan just shared

kenhkan avatar Oct 17 '23 14:10 kenhkan

From 1:1 with Hassan: We need to re-point this after meeting this Friday.

kenhkan avatar Oct 23 '23 18:10 kenhkan

From 1:1 with Hassan: We are leaving this in the backlog to work on something more pressing.

kenhkan avatar Nov 20 '23 13:11 kenhkan

We are very likely going to move away from the 1:1 representation of method:permission beginning with the multichain provider API. I will probably close this issue once we have an issue up for that work.

rekmarks avatar Apr 26 '24 18:04 rekmarks