Skip to content

chore(redirects): add redirects missing in the old docs#2089

Merged
aigerimu merged 3 commits into
mainfrom
closes-2086/old-doc-missed-a-redirect
Apr 17, 2026
Merged

chore(redirects): add redirects missing in the old docs#2089
aigerimu merged 3 commits into
mainfrom
closes-2086/old-doc-missed-a-redirect

Conversation

@novusnota
Copy link
Copy Markdown
Collaborator

@novusnota novusnota commented Apr 16, 2026

Closes #2086

Found a couple related links and added redirects for them too.

Summary by CodeRabbit

  • Documentation

    • Added three permanent redirects so legacy documentation links automatically send users to the updated locations, preserving access to restructured content.
  • Chores

    • Enhanced the link-check process to validate code snippets during broken-link checks, improving detection of invalid references.

@novusnota novusnota requested a review from a team as a code owner April 16, 2026 10:07
@mintlify
Copy link
Copy Markdown

mintlify Bot commented Apr 16, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
mintlify-ton-docs 🟢 Ready View Preview Apr 16, 2026, 10:08 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 16, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 1785ecb7-7f0f-4a70-a137-4528dd63c840

📥 Commits

Reviewing files that changed from the base of the PR and between 85dd0ff and fb01f29.

📒 Files selected for processing (1)
  • package.json
✅ Files skipped from review due to trivial changes (1)
  • package.json

📝 Walkthrough

Walkthrough

Three permanent redirects were added to docs.json and the check:links npm script in package.json was updated to include --check-snippets.

Changes

Cohort / File(s) Summary
Redirect Configuration
docs.json
Added three permanent redirect entries: /learn/overviews/ton-blockchain/start-here, /learn/overviews/cells/foundations/serialization/cells, /learn/overviews/addresses/foundations/addresses/overview.
Scripts
package.json
Updated check:links script from mint broken-links to mint broken-links --check-snippets to enable snippet validation during link checks.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 On old paths I hopped to see,

Three signs point where links should be,
A tiny flag for checks so neat,
Snippets guarded, routes complete.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning The package.json change to add --check-snippets flag is out of scope; it's unrelated to the redirect configuration requested in the linked issue. Remove the --check-snippets modification to package.json as it's outside the scope of adding redirects for old documentation URLs.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change of adding redirect rules for old documentation URLs.
Linked Issues check ✅ Passed The PR addresses the primary objective from issue #2086 by adding the redirect from /learn/overviews/addresses to /foundations/addresses/overview, plus two additional related redirects.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch closes-2086/old-doc-missed-a-redirect

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the updates in docs.json: there are a couple of suggestions to align the new redirects with the intended information architecture, so please apply the inline suggestions.

Comment thread docs.json
Comment thread docs.json
@aigerimu aigerimu merged commit 2343935 into main Apr 17, 2026
9 checks passed
@aigerimu aigerimu deleted the closes-2086/old-doc-missed-a-redirect branch April 17, 2026 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Links] Add a redirect to the old article about addresses

2 participants