-
-
Notifications
You must be signed in to change notification settings - Fork 200
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: integrating signature decoding api #4855
Merged
Changes from 7 commits
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
06b350f
Integrating signature decoding api
jpuri 39cc9ac
Integrating signature decoding api
jpuri 15203de
update
jpuri b4a1ec5
Update
jpuri 0cc8436
Update
jpuri 1edf2f8
Update
jpuri fd39034
Update
jpuri 7553a4e
Update
jpuri fcde11b
Update
jpuri 0b01158
Merge branch 'main' into sign_api
jpuri 6cd0423
Update
jpuri e596371
Merge branch 'sign_api' of github.com:MetaMask/core into sign_api
jpuri 68dc5ff
Update
jpuri 83969e3
update
jpuri 9e25b9a
update
jpuri 1ac68ca
update
jpuri be100d3
Merge branch 'main' into sign_api
jpuri 3c9b7b4
update
jpuri ac2d149
update
jpuri dc7fcb7
update
jpuri dfd539b
update
jpuri d5df2c4
update
jpuri 2cc2d7c
update
jpuri 7fc39cf
Merge branch 'main' into sign_api
jpuri 8ed4da6
update
jpuri 92e752f
update
jpuri 12e91ad
Merge branch 'sign_api' of github.com:MetaMask/core into sign_api
jpuri 964dd73
update
jpuri 329b097
update
jpuri 83f4ede
update
jpuri 9927cca
update
jpuri deb925d
update
jpuri e7cf0cf
Update packages/signature-controller/src/SignatureController.ts
jpuri 0d9818d
update
jpuri 92fa9ea
Merge branch 'sign_api' of github.com:MetaMask/core into sign_api
jpuri 33cb319
Update packages/signature-controller/src/types.ts
jpuri 01d4425
Merge branch 'main' into sign_api
jpuri 4ae4551
update
jpuri adb98cc
update
jpuri 80ff99b
update
jpuri b6b7af4
Merge branch 'main' into sign_api
jpuri 6b100c0
update
jpuri e6cfd86
Merge branch 'sign_api' of github.com:MetaMask/core into sign_api
jpuri 82e3076
update
jpuri 94caf4c
update
jpuri 3edc64d
update
jpuri 4b165e1
update
jpuri e963191
Merge branch 'main' into sign_api
jpuri cdefae9
update
jpuri 7d718f8
Merge branch 'sign_api' of github.com:MetaMask/core into sign_api
jpuri f1d7dc8
update
jpuri 108749f
update
jpuri 64395a3
update
jpuri fae20c7
update
jpuri 658146d
Merge branch 'main' into sign_api
jpuri bd75416
Merge branch 'main' into sign_api
matthewwalsh0 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
export const EthMethod = { | ||
PersonalSign: 'personal_sign', | ||
Sign: 'eth_sign', | ||
SignTransaction: 'eth_signTransaction', | ||
SignTypedDataV1: 'eth_signTypedData_v1', | ||
SignTypedDataV3: 'eth_signTypedData_v3', | ||
SignTypedDataV4: 'eth_signTypedData_v4', | ||
}; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor, this feels more like an enum as it's a fixed set of values to identify a type of something.
Which would also mean it could go in
types.ts
.But we can do this later since it's not exported.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried that but it is giving build issues somehow.