MetaMask / metamask-snaps-beta

Fork of MetaMask that supports plugins! Read the Wiki!
https://github.com/MetaMask/metamask-snaps-beta/wiki
MIT License
144 stars 57 forks source link

Add identity api patch1 #80

Closed danjm closed 5 years ago

danjm commented 5 years ago

This PR updates #63 so that it correctly supports the example in https://github.com/MetaMask/mm-plugin/pull/31

The following changes were made: ec4bc84cf Ensure keyring controller continues to correctly update on state changes and account imports 7d015a55a Allow custom signEthTx method to return signed tx as a hex string in tx controller signTransaction method fe9813ee8 Pass origin as first param to signing methods in accounts controller c962e9c27 Fix error catching in signing methods of accounts controller 7e3e1fabe Extract determination of unique domains for a plugin from getHandlerForAccount to its own method fd6d73518 Normalize addresses in account controller state, for consistency with keyring controller fbe68d024 Update accounts controller remove method to properly clean up accounts in state on removal

Note that this PR has been updated from a previous version that included the addition of an api metamask native prompts. This has now all been moved to #86