Skip to content

Conversation

@aigerimu
Copy link
Contributor

closes #1738

@aigerimu aigerimu marked this pull request as draft January 20, 2026 06:18
@github-actions
Copy link

To fix the formatting issues:

  1. Install necessary dependencises: npm ci
  2. Then, run this command:
npm run fmt:some -- ecosystem/ton-pay/api-reference.mdx ecosystem/ton-pay/overview.mdx ecosystem/ton-pay/payment-integration/payments-react.mdx ecosystem/ton-pay/payment-integration/payments-tonconnect.mdx ecosystem/ton-pay/payment-integration/status-info.mdx ecosystem/ton-pay/payment-integration/transfer.mdx ecosystem/ton-pay/quick-start.mdx ecosystem/ton-pay/ui-integration/button-js.mdx ecosystem/ton-pay/ui-integration/button-react.mdx ecosystem/ton-pay/webhooks.mdx

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 adding the TON Pay button docs; I have several suggestions in ecosystem/ton-pay/ui-integration—please apply the inline suggestions to align the new pages with the style guide.

Comment on lines +1 to +4
---
title: "TonPay button using JS"
sidebarTitle: "Button JS"
---

Choose a reason for hiding this comment

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

[HIGH] Incorrect TON casing in JS button title

The JS button documentation frontmatter title uses TonPay instead of the canonical TON casing, conflicting with the TON-specific casing rules in contribute/style-guide-extended.mdx:1111-1114. This inconsistency can confuse readers and breaks searchability and brand consistency across the TON Pay SDK docs. The React button page shows the same issue, so fixing this title helps keep the entire UI integration group aligned with the TON Pay SDK naming. The primary location here points to the newly added frontmatter lines that introduce the incorrect casing.

Suggested change
---
title: "TonPay button using JS"
sidebarTitle: "Button JS"
---
---
title: "TON Pay button using JS"
sidebarTitle: "Button JS"
---

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

Comment on lines +1 to +4
---
title: "TonPay button using JS"
sidebarTitle: "Button JS"
---
Copy link

@github-actions github-actions bot Jan 20, 2026

Choose a reason for hiding this comment

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

[HIGH] TonPay JS button How-to frontmatter not set

The new TonPay JS button UI integration page uses a generic title ("TonPay button using JS") and sidebarTitle ("Button JS") instead of following the required How to X frontmatter pattern for How‑to pages. According to the style guide’s How‑to frontmatter rule (https://github.com/ton-org/docs/blob/main/contribute/style-guide-extended.mdx?plain=1#L956-L976), How‑to pages must use title: "How to X" and set sidebarTitle to X or a shorter alias of X. This inconsistency harms searchability for “how to …” queries and makes navigation labels less predictable. Aligning the frontmatter with the pattern keeps the How‑to docs coherent and easier to scan in the sidebar.

Suggested change
---
title: "TonPay button using JS"
sidebarTitle: "Button JS"
---
---
title: "How to add TonPay button using JS"
sidebarTitle: "Add TON Pay button using JS"
---

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

Comment on lines +1 to +4
---
title: "TonPay button using React"
sidebarTitle: "Button React"
---

Choose a reason for hiding this comment

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

[HIGH] Incorrect TON casing in React button title

The React button documentation frontmatter title also uses TonPay instead of the prescribed TON casing, again violating the TON-specific casing rule in contribute/style-guide-extended.mdx:1111-1114. This leads to inconsistent branding within the TON Pay SDK docs and can reduce clarity when users navigate or search for TON-related content. The JS button page exhibits the same pattern, so harmonizing both titles with TON Pay SDK ensures consistent terminology across the UI integration group. The location above targets the new frontmatter block that introduces the incorrect casing in this file.

Suggested change
---
title: "TonPay button using React"
sidebarTitle: "Button React"
---
---
title: "TON Pay button using React"
sidebarTitle: "Button React"
---

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

Comment on lines +1 to +4
---
title: "TonPay button using React"
sidebarTitle: "Button React"
---

Choose a reason for hiding this comment

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

[HIGH] TonPay React button How-to frontmatter not set

The new TonPay React button UI integration page also uses a non‑conforming title ("TonPay button using React") and sidebarTitle ("Button React"), instead of the mandated How to X pattern for How‑to content. Per the How‑to frontmatter rule in the extended style guide (https://github.com/ton-org/docs/blob/main/contribute/style-guide-extended.mdx?plain=1#L956-L976), How‑to pages must have title: "How to X" and a sidebarTitle equal to X or a concise alias. This misalignment reduces consistency with other How‑to docs and weakens “how to …” search behavior and sidebar clarity. Updating the frontmatter to match the pattern restores consistency and improves discoverability.

Suggested change
---
title: "TonPay button using React"
sidebarTitle: "Button React"
---
---
title: "How to add TonPay button using React"
sidebarTitle: "Add TonPay button using React"
---

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.

[Ecosystem > TON Pay]

4 participants