Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughWalkthroughAdds block-explorer links to network and mainnet docs, creates a new "Get WARD" guide, restructures the WARD airdrop doc into a "Withdraw WARD" flow with external-wallet instructions and exchange listings (including Korean doc), and a minor text fix in the Trade page. Changes
Sequence Diagram(s)(Skipped — changes are documentation edits and do not introduce new multi-component control flow.) Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
No actionable comments were generated in the recent review. 🎉 🧹 Recent nitpick comments
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@docs/developer-docs/docs/ward/get-ward.md`:
- Around line 5-12: The doc uses both "$WARD" and "WARD" interchangeably; update
the file to standardize on "WARD" (no dollar sign) throughout—replace all
occurrences of the token string "$WARD" in the heading and body (e.g., the
heading "# Get $WARD" and bullet lines mentioning "$WARD") with "WARD", ensure
the sentence "Please note that it's currently not possible to swap WARD in
Warden." remains consistent, and verify other mentions in this file match the
project's canonical "WARD" usage from mainnet-overview.md and ward-airdrop.md.
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In `@docs/help-center/docs/warden-app/ward-airdrop.md`:
- Around line 132-134: There's a typo in the admonition directive: locate the
malformed directive ':::not' used around the note block in ward-airdrop.md and
change it to the correct ':::note' so the note block renders properly; ensure
the opening and closing admonition markers match (':::note' ... ':::') and leave
the inner text unchanged.
- Around line 182-195: Remove or clarify the "Binance Alpha" list item: either
delete the "Binance Alpha" bullet (and its URL) or change its text to explicitly
state it is a Binance Wallet discovery/pre-listing page (not a Binance CEX spot
listing) and add a note that it is not an official Binance exchange spot market
unless an official Binance spot listing announcement for WARD is verified;
update the list entry referencing the "Binance Alpha" anchor text and URL
(0x6dc200b21894af4660b549b678ea8df22bf7cfac) accordingly.
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
docs/developer-docs/docs/korean/ward.md (1)
1-1:⚠️ Potential issue | 🟡 MinorRemove the BOM before the front-matter delimiter.
The leading BOM character can break front‑matter parsing in some Markdown tooling.
🤖 Fix all issues with AI agents
In `@docs/developer-docs/docs/ward/get-ward.md`:
- Around line 14-16: The "Centralized exchanges (spot)" list in get-ward.md
contains an Aster link whose URL path includes "futures", causing a spot vs
futures mismatch; update the Aster entry in the "Centralized exchanges (spot)"
section to either replace the link with Aster's spot trading URL or change the
label to "Centralized exchanges (futures)" so it matches the existing
https://www.asterdex.com/en/trade/pro/futures/WARDUSDT URL, making sure to edit
the Aster list item in docs/developer-docs/docs/ward/get-ward.md accordingly.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@docs/developer-docs/docs/korean/ward.md`:
- Around line 141-150: Verify the Aster and Kraken entries: check the Aster link
and official exchange announcements and remove the "Aster" list item if it does
not officially support WARD (reference the "Aster" bullet), confirm the Kraken
endpoint actually provides a tradable WARD market and update or remove the
"Kraken" bullet accordingly, and annotate the "Binance Wallet (Alpha/Web3)"
entry to include/confirm the BEP-20 contract address
0x6dc200b21894af4660b549b678ea8df22bf7cfac or correct it if wrong.
🧹 Nitpick comments (1)
docs/developer-docs/docs/korean/ward.md (1)
136-142: Aster is a DEX, not a CEX — adjust the classification or sectioning.The “중앙화 거래소” list includes Aster, which presents itself as a decentralized perp DEX. Consider moving Aster under a separate “탈중앙화 거래소” section or renaming the heading to avoid misleading readers. (asterdex.io)
Suggested restructure
-**중앙화 거래소** -- [Aster](https://www.asterdex.com/en/trade/pro/futures/WARDUSDT) +**중앙화 거래소** +... + +**탈중앙화 거래소** +- [Aster](https://www.asterdex.com/en/trade/pro/futures/WARDUSDT)
1 participant
Uh oh!
There was an error while loading. Please reload this page.