Skip to content

feat(ton-connect): change icon for ton connect#442

Merged
heyllog merged 1 commit into
mainfrom
feat/square-svg
Jun 1, 2026
Merged

feat(ton-connect): change icon for ton connect#442
heyllog merged 1 commit into
mainfrom
feat/square-svg

Conversation

@heyllog
Copy link
Copy Markdown
Contributor

@heyllog heyllog commented Jun 1, 2026

Summary by CodeRabbit

  • Style
    • Updated the TON token icon asset format for consistent display across the application.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 1, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 249fa3c4-72e7-43c7-a00a-5921b137d198

📥 Commits

Reviewing files that changed from the base of the PR and between edb0390 and df51301.

⛔ Files ignored due to path filters (1)
  • apps/demo-wallet/public/ton.svg is excluded by !**/*.svg
📒 Files selected for processing (2)
  • apps/demo-wallet/src/components/swap/TokenSelector.tsx
  • apps/demo-wallet/src/hooks/useJettonInfo.ts

📝 Walkthrough

Walkthrough

This PR updates the TON token logo asset format across the demo wallet from SVG to PNG. Both the TON_INFO metadata constant and the TokenSelector token info computation now reference /ton.png instead of /ton.svg.

Changes

TON Logo Format Update

Layer / File(s) Summary
TON logo asset path updates
apps/demo-wallet/src/hooks/useJettonInfo.ts, apps/demo-wallet/src/components/swap/TokenSelector.tsx
TON_INFO exported metadata and TokenSelector's native token info computation both switch the logo reference from /ton.svg to /ton.png.

🎯 1 (Trivial) | ⏱️ ~2 minutes

🐰 A rabbit hops through the vault,
Where logos once were spiky SVG,
Now smooth PNG files roll along,
The TON token wears its new square coat,
And renders bright and swift! 🎨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main change: updating the TON token icon from SVG to PNG format across the demo wallet components.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/square-svg

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@heyllog heyllog merged commit 2df7aae into main Jun 1, 2026
5 checks passed
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.

3 participants