-
Notifications
You must be signed in to change notification settings - Fork 18
EVM: Split signer from account #40
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
Merged
Merged
Changes from all commits
Commits
Show all changes
59 commits
Select commit
Hold shift + click to select a range
5805196
WIP: poc-signer
claudiovb 8a4dc0c
WIP: poc-signer
claudiovb e04a862
add ISignerEvm
claudiovb 4c77e8c
add ISignerEvm
claudiovb 4f18ce8
feat: add ledger signer evm
sontuphan 2bd8427
fix: lint standardjs
sontuphan c00a22e
fix: add deps
sontuphan fc31010
fix: remote master node in bip44 derivation path in ledger signer
sontuphan 6dbfe99
fix: add private async connect
sontuphan 5af6f0a
Merge pull request #1 from sontuphan/poc-ledger-signer
claudiovb 3a6b289
fix signer
claudiovb 1734267
add safety checks to ledger signer
claudiovb bb9f608
fix sendTransaction
claudiovb ccdcfb9
fix derive function
claudiovb c4a879f
Merge branch 'tetherto:main' into poc-signer
claudiovb db2ff81
clean up for publishing
claudiovb f921284
update readme code
claudiovb f9fb077
refactor ledger submodule
claudiovb d937ab8
remove not used signer config object
claudiovb 123e995
add missing method to ledger signer
claudiovb b2d6660
fix: bump dmk versions, fix minor bugs, autogen signer types
sontuphan d03a102
Merge pull request #3 from sontuphan/fix/poc-signer
claudiovb b901da8
fix: remove isActive and update ledger dispose
sontuphan 9372233
Plain object signers fix, update packages
claudiovb 8c12540
Merge changes
claudiovb cbc2d6c
Merge changes
claudiovb 84b8cb1
fixes
claudiovb 86d0061
Update src/signers/seed-signer-evm.js
claudiovb 4b03dc9
Update src/signers/ledger-signer-evm.js
claudiovb 997cbb5
defaults in the description not in brackets
claudiovb 9f94876
Merge branch 'poc-signer' of github.com:claudiovb/wdk-wallet-evm into…
claudiovb cefe0a9
Small JSDOC fixes
claudiovb a213e11
Split test for signers & ISignerEVM contract fullfilment
claudiovb 6a378b8
Merge with new updates
claudiovb bf11299
Fix JSDOCS issues
claudiovb 3a041e0
Fix Ledger signing and unneeded guard clauses
claudiovb 65576ed
Fix tests comments
claudiovb 432aac7
Update types
claudiovb e615962
Update types and JSdocs small polishes
claudiovb 109ba85
merge changes from main
claudiovb d979205
refactor to conform with new wdk-wallet implementation
claudiovb eb47d2c
fix bug from ricardo comment
claudiovb d81a6d0
update not needed tests and jsdoc definitions
claudiovb aeb52e3
minor fixes
claudiovb 64ea4b9
update derive so its compatible with baseClass, refactor for better d…
claudiovb 03cba9d
update package to published wdk-wallet version
claudiovb 13cf4ac
don't pin peerdeps to avoid issue on consumers that use ledger instal…
claudiovb e4a0235
merge with changes on main
claudiovb 3a2e0cd
address Ricardo's comments
claudiovb cfe141e
update with main
claudiovb a264337
update to to be nullable for allowing sc creation
claudiovb 0dcd6a8
add test to validate smart contract creation
claudiovb 49a3f3e
remove ledger from this branch
claudiovb 74b3ea9
add fromPrivateKey static method to EVM
claudiovb 6bc4b85
update wdk-wallet dep type
claudiovb 815d022
update JSDOC
claudiovb 853525f
remove useless guard on getAddress
claudiovb 79c41bd
remove stale ts text
claudiovb ad224c6
update wdk-wallet to latest version
claudiovb 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 hidden or 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 |
|---|---|---|
| @@ -1,3 +1,4 @@ | ||
| node_modules | ||
| coverage | ||
| .vscode | ||
| .DS_Store | ||
| .vscode |
This file contains hidden or 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 hidden or 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.
Uh oh!
There was an error while loading. Please reload this page.