added info about handling web wallet #388
Open
Conversation
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
rohit-710
reviewed
May 5, 2025
| We recommend closing the web wallet tab after the user approves the request and redirecting back to the dApp. For web requests, redirect to the original browser tab, and for mobile requests, redirect to the native mobile dApp. | ||
|
|
||
| <Info> | ||
| To test your web wallet connection flow, use our [Appkit Laboratory](https://appkit-lab.reown.com/) by adding a custom wallet with your web wallet URL. |
There was a problem hiding this comment.
@vrajdesai78 can we add the steps that one needs to take to add a custom wallet to AppKit lab? I know it is very straight forward but this isn't always obvious to lot of our customers.
skibitsky
reviewed
May 5, 2025
| 2. Implement a scanner to read the WC URI from a WalletConnect QR code | ||
| 3. Allow users to manually enter the WC URI in an input field | ||
|
|
||
| When opening the wallet directly from the dApp, the `WC_URI` will be in the following format in the query parameters: |
There was a problem hiding this comment.
Also need to add a similar paragraph about session requests. The UR for session requests is {YOUR_WALLET_URL}/wc?requestId={requestId}&sessionTopic={session.Topic}
skibitsky
reviewed
May 5, 2025
| {YOUR_WALLET_URL}/wc?uri={WC_URI} | ||
| ``` | ||
|
|
||
| We recommend closing the web wallet tab after the user approves the request and redirecting back to the dApp. For web requests, redirect to the original browser tab, and for mobile requests, redirect to the native mobile dApp. |
There was a problem hiding this comment.
Explain how to redirect to the native app? The link to the native app may be available inside of peer metadata, otherwise, they should probably show "go back to the app" message.
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
Enter the description of this PR and add any important context/info.
Tests