Skip to content

Conversation

@verytactical
Copy link
Collaborator

@verytactical verytactical commented Oct 28, 2025

Closes #107

@anton-trunov anton-trunov merged commit b43b39b into main Oct 28, 2025
4 checks passed
@anton-trunov anton-trunov deleted the dedup-system branch October 28, 2025 17:25
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 update. I left several suggestions in foundations/system.mdx; please apply the inline suggestions.

</Aside>

<Aside>
System contracts are smart contracts and have on‑chain addresses. See [config parameters 1–4](https://tonviewer.com/config). The Config account stores the Config contract address. To track changes, review proposals to the Config contract.

Choose a reason for hiding this comment

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

[HIGH] External explorer links used instead of internal docs

The page links to an external explorer for configuration parameters instead of the internal canonical docs, violating the internal docs preference. The diff adds such an external link at L12. The same pattern appears elsewhere in the changed file (e.g., foundations/system.mdx L27, L50, L52, L104) and should be updated similarly.

Suggested change
System contracts are smart contracts and have on‑chain addresses. See [config parameters 1–4](https://tonviewer.com/config). The Config account stores the Config contract address. To track changes, review proposals to the Config contract.
System contracts are smart contracts and have on‑chain addresses. See [config parameters 1–4](/foundations/config). The Config account stores the Config contract address. To track changes, review proposals to the Config contract.

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

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.

[Foundations > System]

3 participants