feat(tari_wallet_daemon): walletconnect support #1083
Merged
Conversation
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
mrnaveira
changed the title
This was referenced Jul 17, 2024
mrnaveira
changed the title
…into walletconnect
sdbondi
previously approved these changes
Jul 25, 2024
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
…into walletconnect
…into walletconnect
…into walletconnect
sdbondi
approved these changes
Jul 30, 2024
Merged
via the queue into
tari-project:development
with commit Jul 30, 2024
5b0820e
12 checks passed
github-merge-queue bot
pushed a commit
to tari-project/tari.js
that referenced
this pull request
Aug 7, 2024
Description --- * Adds a new provider for WalletConnect * Removed the `example` web page, as we already have [a project or that purpose](https://github.com/tari-project/tari-template-web). Motivation and Context --- See tari-project/tari-dan#1083 and tari-project/tari-template-web#6 for more context How Has This Been Tested? --- This can only be tested alongside the changes in the tari wallet daemon to support WalletConnect: tari-project/tari-dan#1083 What process can a PR reviewer use to test or verify this change? --- See previous section Breaking Changes --- - [x] None - [ ] Requires data directory on base node to be deleted - [ ] Requires hard fork - [ ] Other - Please specify --------- Co-authored-by: Stan Bondi <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Adds support for WalletConnect in the
tari_wallet_daemon_ui:VITE_WALLET_CONNECT_PROJECT_IDMotivation and Context
We want to support WalletConnect in Tari, as this is a de facto standard for web-to-wallet interaction. It will allow any wallet (web or mobile) to add support for Tari without us having to implement anything extra.
In this context, the goal of this PR is to add support for WalletConnect in the Tari Wallet Daemon alongside the exising TariConnector.
For now we have separated UI buttons for both TariConnector and WalletConnect, due do some TariConnect logic being entangled into the UI. It should be possible to refactor to only have one generic button that accepts both types of connection URIs but it was left out of the scope of this PR.
See tari-project/tari.js#27 and tari-project/tari-template-web#6 for more information.
How Has This Been Tested?
Manually by running
tari_swarm, opening the tari wallet daemon web UI and then paste a valid WalletConnect URIWhat process can a PR reviewer use to test or verify this change?
See previous section
Breaking Changes