feat: Draft implementation of caveat merging #24271

rekmarks · 2024-04-29T05:18:59Z

A draft implementation of caveat merging using MetaMask/core#4222 and some yet to be published work.

PR template

Description

Fixes:

I’ve followed MetaMask Coding Standards.
I've completed the PR template to the best of my ability
I’ve included tests if applicable
I’ve documented my code using JSDoc format if applicable
I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

rekmarks · 2024-05-07T00:14:21Z

app/scripts/controllers/permissions/background-api.js

-      if (existingAccounts.includes(account)) {
-        return;
-      }
+  const addMoreAccounts = (origin, accountOrAccounts) =>


TODO: Check the diff, console.error() on empty.

feat: Draft implementation of caveat merging

0de5c61

rekmarks commented May 7, 2024

View reviewed changes