-
Notifications
You must be signed in to change notification settings - Fork 102
Docs Update: Fixing broken redirects on WalletKit technical docs #613
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
base: main
Are you sure you want to change the base?
Conversation
@devin-ai-integration[bot] Please review the tone of voice for the content changes in this PR against Reown's brand guidelines. 📝 Content Review RequestFiles to review: 2
Review focus:
Guidelines summary:
Please analyze the content changes and provide constructive feedback on tone and voice alignment. |
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.
Pull Request Overview
This PR fixes broken redirects in WalletKit technical documentation by correcting malformed internal links and removing redundant text.
- Removes
.mdx
file extension from internal link reference - Eliminates broken self-referential link with grammatical fix
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
File | Description |
---|---|
walletkit/web/chain-abstraction.mdx | Fixes internal link by removing .mdx extension from anchor reference |
walletkit/features/chain-abstraction.mdx | Removes broken self-referential link and fixes grammatical error |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
### What are the limitations? | ||
|
||
We currently support 1:1 transfers i.e. sourcing funds from one address to another. Make sure that you're transferring minimum 0.6$ worth of tokens(/walletkit/../walletkit/features/early-access/chain-abstraction.mdx#what-are-the-supported-tokens-and-networks) and have enough gas to pay bridging fees. | ||
We currently support 1:1 transfers i.e. sourcing funds from one address to another. Make sure that you're transferring minimum 0.6$ worth of token and have enough gas to pay bridging fees. |
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.
Grammar issue: 'token' should be 'tokens' to maintain consistency with the plural form used elsewhere in the documentation.
We currently support 1:1 transfers i.e. sourcing funds from one address to another. Make sure that you're transferring minimum 0.6$ worth of token and have enough gas to pay bridging fees. | |
We currently support 1:1 transfers i.e. sourcing funds from one address to another. Make sure that you're transferring minimum 0.6$ worth of tokens and have enough gas to pay bridging fees. |
Copilot uses AI. Check for mistakes.
@devin-ai-integration[bot] Please review the tone of voice for the content changes in this PR against Reown's brand guidelines. 📝 Content Review RequestFiles to review: 6
Review focus:
Guidelines summary:
Please analyze the content changes and provide constructive feedback on tone and voice alignment. |
Description
This PR fixes some of the broken redirects inside the technical docs of WalletKit.
Tests
Direct link to the deployed preview files