Add `wallet_requestPermissions` method #21

felipelincoln · 2022-08-05T09:51:31Z

Was implemented following the standards from RPC API | MetaMask Docs and EIP-2255: Wallet Permissions System

jessgusclark

Thanks for contributing! This looks great, one small change to reduce duplicate code.

jessgusclark · 2022-08-08T11:22:46Z

src/index.ts

+          return new Promise((resolve, reject) => {
+            this.acceptEnable = resolve
+            this.rejectEnable = reject
+          }).then(() => permissions)


Can we combine this with the one from eth_accounts (line 71) so we don't have repeated code?

Perhaps something like this:

private handleManualConfirm(response: any): Promise<any> { // create the promise and then return the response }

That will make it easier to extend the confirm/reject to other methods.