chore: replace some redirects to old-docs.ton.org with existing pages#2178
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
📝 WalkthroughWalkthroughThis PR migrates documentation away from old-docs.ton.org by updating MDX pages to use improved callout components and reconfiguring redirect rules in docs.json. Content pages convert blockquote warnings to structured ChangesDocumentation Migration: Aside Components and Legacy Redirect Updates
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add 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 |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
standard/tokens/jettons/comparison.mdx (1)
42-48: ⚡ Quick winConsider adding a title to improve scannability.
The
Asidecomponent in the style guide example (contribute/style-guide.mdx lines 80-82) includes atitleprop for improved scannability. Adding a descriptive title here (e.g.,title="Verify jetton authenticity"ortitle="Security warning") would help readers quickly identify the warning type when scanning the page.♻️ Suggested enhancement
<Aside type="caution" + title="Verify jetton authenticity" > On TON, multiple jettons can share the same name — a counterfeit jetton may look identical to the real USDT but have no value. Verify legitimacy by confirming [USDT's jetton master address](https://tonviewer.com/EQCxE6mUtQJKFnGfaROTKOt1lZbDiiX1kCixRv7Nw2Id_sDs?section=jetton).As per coding guidelines, "Keep it scannable. Use short sections and paragraphs, clear headings, lists, and tables. Most readers skim to find the next action."
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@standard/tokens/jettons/comparison.mdx` around lines 42 - 48, The Aside component used for the caution block lacks a title prop which harms scannability; update the Aside instance (the Aside component around the jetton warning) to include a descriptive title such as title="Verify jetton authenticity" or title="Security warning" so readers can immediately identify the note type when scanning the page, ensuring the prop is added to the same Aside element and matches styling conventions used elsewhere in the style guide examples.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@standard/tokens/jettons/comparison.mdx`:
- Around line 42-48: The Aside component used for the caution block lacks a
title prop which harms scannability; update the Aside instance (the Aside
component around the jetton warning) to include a descriptive title such as
title="Verify jetton authenticity" or title="Security warning" so readers can
immediately identify the note type when scanning the page, ensuring the prop is
added to the same Aside element and matches styling conventions used elsewhere
in the style guide examples.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: e27a9abb-5628-416e-839d-c526edd9fa60
📒 Files selected for processing (4)
contribute/style-guide.mdxdocs.jsonstandard/tokens/jettons/comparison.mdxstandard/wallets/v4.mdx
Closes #2177
Summary by CodeRabbit