Skip to content

Conversation

memearchivarius
Copy link
Collaborator

Closes #42

I've transferred some details from old documentation, it is still a draft.

I've seen /pending/reference/blockchain/ with more in-depth pages, should I change this page to be a starting point for those pending pages?

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as duplicate.

Copy link
Collaborator

@novusnota novusnota left a comment

Choose a reason for hiding this comment

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

Some high-level suggestions

This comment was marked as duplicate.

This comment was marked as duplicate.

1 similar comment

This comment was marked as outdated.

This comment was marked as resolved.

@novusnota
Copy link
Collaborator

/review

novusnota
novusnota previously approved these changes Oct 6, 2025
Copy link

github-actions bot commented Oct 6, 2025

Thanks for the update. I took a look at the docs change in ton/phases-and-fees.mdx; a small fix is needed to address a broken internal link.

Findings (1)

High (1)

[HIGH] Broken internal link to non-existent “message modes cookbook”

Location: https://github.com/tact-lang/mintlify-ton-docs/blob/52ee7d11f2306827f073541ae76c958de9391995/ton/phases-and-fees.mdx?plain=1#L374-L374

Description:
The added bullet links to /ton/message-modes, but no file or route in the repo matches this path, so the link is broken. This violates the style guide requirement that internal anchors resolve. A stable in-repo alternative is the TVM instructions reference documenting message mode semantics.

Suggestion:

- - Refer to the [message modes cookbook](/ton/message-modes) for additional modes.
+ - See [`SENDMSG`](/tvm/instructions) for message mode semantics.
- - Refer to the [message modes cookbook](/ton/message-modes) for additional modes.
+ - Refer to [TVM instructions](/tvm/instructions) for message mode details.

@memearchivarius
Copy link
Collaborator Author

@anton-trunov resolved

Copy link

To fix the formatting issues:

  1. Install necessary dependencies: npm ci
  2. Then, run this command:
npx remark -o --silent --silently-ignore foundations/fees.mdx foundations/phases.mdx patterns/fee-management.mdx ecosystem/blueprint/benchmarks.mdx 

anton-trunov
anton-trunov previously approved these changes Oct 17, 2025
@verytactical verytactical merged commit 71f9a62 into main Oct 17, 2025
4 checks passed
@verytactical verytactical deleted the 42-phases-and-fees branch October 17, 2025 12:00
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 > Execution phases and fees]

5 participants