Skip to content

V26/6/1#678

Open
nstus wants to merge 2 commits into
v26/6/0from
v26/6/1
Open

V26/6/1#678
nstus wants to merge 2 commits into
v26/6/0from
v26/6/1

Conversation

@nstus

@nstus nstus commented Jun 22, 2026

Copy link
Copy Markdown
Collaborator

Note

Low Risk
Display and translation-only changes with protocol identifiers still hardcoded as TON; large locale churn but low functional risk beyond wording consistency.

Overview
Refines the Gram rebrand so display copy depends on UI language and the transitional “old name” labels are used only where intended.

Brand config splits coinName and coinSymbolWithEx into English (prev. …) vs non-English (ex-…) fields, with getBrandCoinName() / getBrandCoinSymbolWithEx() resolving from the active language. Apps pass i18n.language (or the extension UI language) into useTWithReplaces so %{coinName} interpolates correctly at runtime.

Locale strings across tonkeeper-web swap most %{coinSymbolWithEx} placeholders to %{coinSymbol} (plain GRAM), so fee hints, multisend, Pro renewal, and similar UI no longer show the long transitional ticker. The extension build bakes per-folder coinName / coinSymbolWithEx into _locales via the new helpers.

Core behavior: getJettonSymbol for native TON now returns the pure ticker; TON_ASSET.name defaults to the English coin name where core has no language context (UI should use language-aware hooks for names).

Reviewed by Cursor Bugbot for commit 7d20e21. Bugbot is set up for automated code reviews on this repo. Configure here.

@nstus nstus changed the base branch from main to v26/6/0 June 22, 2026 13:42
@github-actions

Copy link
Copy Markdown

Successful WEB deployment 🚀🚀🚀

Well done!
Link to test environment:
https://eb9aa886.tonkeeper-web.pages.dev

@nstus

nstus commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 7d20e21. Configure here.

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.

1 participant