Skip to content

Add more TRC-20 tokens.#6932

Merged
PrisionMike merged 1 commit into
mainfrom
prisionmike/update-trc20-list
May 15, 2026
Merged

Add more TRC-20 tokens.#6932
PrisionMike merged 1 commit into
mainfrom
prisionmike/update-trc20-list

Conversation

@PrisionMike PrisionMike requested review from ibz and romanz May 15, 2026 05:26
@PrisionMike PrisionMike requested a review from obrusvit as a code owner May 15, 2026 05:26
@trezor-bot trezor-bot Bot added this to Firmware May 15, 2026
@github-project-automation github-project-automation Bot moved this to 🔎 Needs review in Firmware May 15, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 15, 2026

en main(all)

model device_test click_test persistence_test
T2T1 test(all) main(all) test(all) main(all) test(all) main(all)
T3B1 test(all) main(all) test(all) main(all) test(all) main(all)
T3T1 test(all) main(all) test(all) main(all) test(all) main(all)
T3W1 test(all) main(all) test(all) main(all) test(all) main(all)

Latest CI run: 25903145659

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 15, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 758bdd93-70b5-419a-b99f-4ca36fc4ca9d

📥 Commits

Reviewing files that changed from the base of the PR and between 00f5eb2 and 26fd00d.

📒 Files selected for processing (1)
  • core/src/apps/tron/consts.py
🚧 Files skipped from review as they are similar to previous changes (1)
  • core/src/apps/tron/consts.py

Walkthrough

This pull request updates core/src/apps/tron/consts.py by adding six new TRC-20 token entries (WBT, WTRX, SUNOLD, AINFT, sTRX, Klever). It introduces 0x41-prefixed contract address byte constants, adds corresponding debug base58.encode_check assertions, and extends token_iterator() to yield each token's contract identifier bytes, decimals, and ticker. No other logic or public declarations were changed.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ❓ Inconclusive The description lists the six tokens being added with TRONSCAN links but does not follow the repository's PR template structure for core developers. Add required template sections: assign yourself, add to Firmware project with Priority/Team/Sprint, and set PR status to 'Needs Review'.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: adding more TRC-20 tokens to the codebase.
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 prisionmike/update-trc20-list

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.

@PrisionMike PrisionMike force-pushed the prisionmike/update-trc20-list branch from ff3380f to 00f5eb2 Compare May 15, 2026 06:00
@PrisionMike PrisionMike force-pushed the prisionmike/update-trc20-list branch from 00f5eb2 to 26fd00d Compare May 15, 2026 06:04
@PrisionMike PrisionMike merged commit 6b25525 into main May 15, 2026
110 checks passed
@PrisionMike PrisionMike deleted the prisionmike/update-trc20-list branch May 15, 2026 06:50
@PrisionMike PrisionMike moved this from 🔎 Needs review to ✅ Done (no QA) in Firmware May 15, 2026
@trezor-bot trezor-bot Bot moved this from ✅ Done (no QA) to 🤝 Needs QA in Firmware May 15, 2026
@PrisionMike PrisionMike moved this from 🤝 Needs QA to ✅ Done (no QA) in Firmware May 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done (no QA)

Development

Successfully merging this pull request may close these issues.

2 participants