Skip to content

Conversation

@fakela
Copy link
Contributor

@fakela fakela commented Oct 13, 2025

Closes #702

@github-actions
Copy link

Thanks for the PR. I reviewed the changes across the touched files; I found no issues and no fixes are needed.

@github-actions
Copy link

To fix the formatting issues:

  1. Install necessary dependencies: npm ci
  2. Then, run this command:
npx remark -o --silent --silently-ignore ton/overview.mdx 

@github-actions
Copy link

To fix the formatting issues:

  1. Install necessary dependencies: npm ci
  2. Then, run this command:
npx remark -o --silent --silently-ignore ton/overview.mdx 

@github-actions
Copy link

To fix the formatting issues:

  1. Install necessary dependencies: npm ci
  2. Then, run this command:
npx remark -o --silent --silently-ignore ton/overview.mdx 

@anton-trunov
Copy link
Collaborator

@fakela What issue does this PR resolve? Please make sure all PRs have their corresponding issues. Please re-read #314, explaining our workflow.

@anton-trunov anton-trunov marked this pull request as draft October 13, 2025 11:56
@fakela
Copy link
Contributor Author

fakela commented Oct 13, 2025

@fakela What issue does this PR resolve? Please make sure all PRs have their corresponding issues. Please re-read #314, explaining our workflow.

I'll create an issue and link to it.

@fakela fakela linked an issue Oct 13, 2025 that may be closed by this pull request
@fakela fakela marked this pull request as ready for review October 13, 2025 13:52
@fakela fakela self-assigned this Oct 13, 2025
Copy link
Collaborator

@verytactical verytactical left a comment

Choose a reason for hiding this comment

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

ton/overview.mdx article doesn't match the description in the ticket, and was merged by mistake

"List of article names" is a kind of article from old docs that we intentionally tried to remove.

There isn't much point in fixing wording here.

@verytactical verytactical marked this pull request as draft October 13, 2025 21:34
@fakela
Copy link
Contributor Author

fakela commented Oct 13, 2025

ton/overview.mdx article doesn't match the description in the ticket, and was merged by mistake

"List of article names" is a kind of article from old docs that we intentionally tried to remove.

There isn't much point in fixing wording here.

@reveloper here is the comment

@anton-trunov anton-trunov requested a review from Shvandre October 30, 2025 19:39
Copy link
Collaborator

@Shvandre Shvandre left a comment

Choose a reason for hiding this comment

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

LGTM

@Shvandre
Copy link
Collaborator

But why this PR closes already closed issue, which is closed as duplicate of another issue....

@reveloper reveloper changed the title update the information on the ton overview page fix: update the information on the ton overview page Nov 10, 2025
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.

[Blockchain Foundations > Overview]

5 participants