-
Notifications
You must be signed in to change notification settings - Fork 0
crosslink Tvm PDF #625
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
crosslink Tvm PDF #625
Conversation
Thanks for the updates to the TON/TVM docs. A couple of high‑severity fixes are needed to align links with our style and ensure readers land on authoritative, precise sources. Findings (2)High (2)[HIGH] Conflated link text and incorrect anchor targetLocation: Line 565 in 873a171
Description: Suggestion: -As described in [1, 2.5.14](#reference-1), all the data used by the TON Blockchain, including the blocks themselves and the blockchain state, can be represented—and are represented—as collections, or “bags”, of cells.
+As described in [1](#reference-1) and [2.5.14 — Everything is a bag of cells](/ton/ton#2-5-14-everything-is-a-bag-of-cells-philosophy), all the data used by the TON Blockchain, including the blocks themselves and the blockchain state, can be represented—and are represented—as collections, or “bags”, of cells. [HIGH] External non‑authoritative PDF link where official local copy existsLocation: Line 4518 in 873a171
Description: Suggestion: -<a id="reference-1"></a> [1] N. Durov, [*Telegram Open Network*](https://www.editionmultimedia.fr/wp-content/uploads/2019/10/Telegram-Open-Network-2017.pdf), 2017.
+<a id="reference-1"></a> [1] N. Durov, [*Telegram Open Network*](/resources/pdfs/ton.pdf), 2017. |
PS: Please ignore github actions comment as it doesn't fit current scope |
ton/tvm.mdx
Outdated
## Abstract | ||
|
||
The aim of this text is to provide a description of the Telegram Open Network Virtual Machine (TON VM or TVM), used to execute smart contracts in the TON Blockchain. | ||
The aim of this text is to provide a description of the Telegram Open Network Virtual Machine (TON VM or TVM), used to execute smart contracts in the TON Blockchain[³³](#footnote-33). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use <Aside>
or <Warning>
. Scrolling down to footnote is error-prone and implemented incorrectly in mintlify.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@reveloper I am changing this from footer to using <Aside>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.

@verytactical The <Aside>
added for TVM
…-ton-docs into crosslink-tvm
As far as I know there was a previous discussion (that didn't involve me) that there should be an example of crosslinks first, and after that we'd expand it to the rest of whitepapers. Let's postpone any changes here until we agree on the way crosslinks should be done, and #627 is merged. |
This whitepaper has only one reference cited:
[1]: N. Durov, “Telegram Open Network,” 2017
This reference is linked in only three sections:
I have attached the following external link to the reference entry N. Durov, “Telegram Open Network,” 2017 for easier access:
https://www.editionmultimedia.fr/wp-content/uploads/2019/10/Telegram-Open-Network-2017.pdf
The 2017 PDF is not hosted on Mintlify (only the 2020 version is) hence the need for the external link. Also, this pdf does not crosslink with other whitepapers currently available on Mintlify.
Closes issue: #560