forked from jl777/SuperNET
-
Notifications
You must be signed in to change notification settings - Fork 98
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(walletconnect): walletconnect client #2183
Closed
Closed
Changes from 28 commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
ff267a4
make walletconnect lib compile
borngraced 3a585a1
Cargo.lock
borngraced 60638bd
Merge remote-tracking branch 'origin/dev' into walletconnect
borngraced de9b42a
use thread_rng in generate_mnemonic for now
borngraced 7d2ef1e
save dev state
borngraced 1712c08
revert bip39 dep changes
borngraced 38f8e0f
port and implement walletconnect wss connection options
borngraced 71feee0
save dev state
borngraced 75db037
save dev state - native client impl
borngraced 9d45139
implement walletconnect client with unit tests
borngraced a6a53e5
commit lock file
borngraced 4e7f633
fix unit test wasm
borngraced a4d827b
polish and minor changes
borngraced 59050d6
save dev state
borngraced c87d9a3
rename module
borngraced 615d4cb
fix client bug and refactoring
borngraced 35ab1e3
fix merge conflicts
borngraced 3b98328
resolve deps
borngraced 902bc7b
debug deps
borngraced 3957935
rename mod and add port info
borngraced e84db08
commit lock file and Cargo.toml
borngraced e6093fd
use MmError handling
borngraced ec3a3fb
minor refactorings
borngraced 1649a7a
fix formattiing, update dalek deps to 4.1.3, update lib docs
borngraced 44f847c
debug curve25519_dalek_backend
borngraced 5027a90
debug curve25519_dalek_backend build 2
borngraced 978c548
update rand and revert generate_mnemonic
borngraced 0028369
error improv and other minor changes
borngraced 0a24364
revert editor format
borngraced 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
Oops, something went wrong.
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.
Is it your editor making these changes automatically, or are you making them intentionally?
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.
that's my editor.. sorry 🙁
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.
The formatting is fine, it's just too much out-of-context change for a feature PR in my opinion
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.
...and the formatting in some Cargo.toml files aren't consistent enough