-
Notifications
You must be signed in to change notification settings - Fork 4
feat: tonviewer wording/images #576
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
Conversation
|
To fix the formatting issues:
npx remark -o --quiet --silently-ignore ecosystem/explorers/tonviewer.mdx |
|
To fix the formatting issues:
npx remark -o --quiet --silently-ignore ecosystem/explorers/tonviewer.mdx |
|
To fix the formatting issues:
npx remark -o --silent --silently-ignore ecosystem/explorers/tonviewer.mdx |
|
Thanks for the updates to the TON Viewer docs. A few high‑severity style and link integrity fixes are needed before merge, all in ecosystem/explorers/tonviewer.mdx. Findings (3)High (3)[HIGH] Broken internal link to Blocks referenceDescription: Suggestion: - - [Blocks](/ton/blocks) — containers of transactions. In explorers, they expose block metadata and configuration parameters, allowing you to trace activity and understand how the blockchain operates.
+ - [Blocks](/ton/tblkch) — containers of transactions. In explorers, they expose block metadata and configuration parameters, allowing you to trace activity and understand how the blockchain operates.[HIGH] UI label not quoted per house ruleDescription: Suggestion: - - Use **Show details** to examine full transaction and message information.
+ - Use "Show details" to examine full transaction and message information.[HIGH] Silent address truncationDescription: Suggestion: - At point **A** (address `UQDj…D0lN`), the user’s wallet sends an external-in message to _transfer an NFT_.
+ At point **A** (address `UQDj…D0lN` (truncated)), the user’s wallet sends an external-in message to _transfer an NFT_.
- - B — the NFT contract at address `EQCo…gJdV`, validates ownership and executes the transfer.
+ - B — the NFT contract at address `EQCo…gJdV` (truncated), validates ownership and executes the transfer. |
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.
- fix merge conflicts
- fix AI review
- screenshots for light color theme needed
- make the file names for dark and light theme screenshots consistent with the rest of the file names
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.
Thanks for the PR. There’s one inline suggestion in ecosystem/explorers/tonviewer.mdx:; please apply it.
|
/review |
|
Re-deployed: https://companyname-a7d5b98e-tonviewer.mintlify.app/ |
|
We'll have to make a total rewrite of this article in the future. |
closes #572
closes #573
closes #574
closes #613