Skip to content

Conversation

@aigerimu
Copy link
Contributor

closes #1577

Copy link

@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 here; I have one inline suggestion in docs.json: please apply the inline suggestion before merging.

Comment on lines 446 to 451
"group": "Migrating from FunC",
"pages": [
"languages/tolk/from-func/tolk-vs-func",
"languages/tolk/from-func/tolk-vs-tlb",
"languages/tolk/from-func/stdlib-fc",
"languages/tolk/from-func/converter"
]

Choose a reason for hiding this comment

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

[HIGH] Deleted page leaves broken internal link

In docs.json the “Migrating from FunC” group no longer includes the languages/tolk/from-func/tolk-vs-tlb page in its pages array (L446–L451), while languages/tolk/from-func/tolk-vs-tlb.mdx has been deleted in this PR. However, languages/tolk/idioms-conventions.mdx still links to /languages/tolk/from-func/tolk-vs-tlb at https://github.com/ton-org/docs/blob/9f45aa2b4409ee09b41baca53fdb28d698b66a3a/languages/tolk/idioms-conventions.mdx?plain=1#L35-L41, so the link now targets a page that is no longer registered in the navigation or present in the tree. This turns a previously valid internal link into a broken target, violating the style guide’s global override against broken or missing anchors. Because the style guide treats broken internal links as always HIGH severity, this remains a HIGH issue.

Please leave a reaction 👍/👎 to this suggestion to improve future reviews for everyone!

@tolk-vm
Copy link
Collaborator

tolk-vm commented Dec 10, 2025

Absolutely wrong. See a comment in #1577

Copy link

@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.

No documentation issues detected.

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.

[Tolk > Migrating from FunC > Tolk-vs-TL-B] Remove article from documentation

3 participants