-
Notifications
You must be signed in to change notification settings - Fork 92
feat: add Tron chain definitions for currency package #1688
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
base: feat/tron-smart-contracts
Are you sure you want to change the base?
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the
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. Comment |
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Greptile OverviewGreptile SummaryAdded comprehensive support for Tron blockchain to the currency package, enabling the system to handle Tron mainnet and Nile testnet chains.
The implementation follows the existing architectural patterns and is consistent with how other blockchain networks are integrated into the system. Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant App as Application
participant TronChains as TronChains
participant ChainsAbstract as ChainsAbstract
participant TronDef as Tron/Nile Definitions
participant Types as Currency Types
App->>Types: Import TronChainName type
App->>TronChains: Import TronChains instance
TronChains->>TronDef: Load chain definitions (tron, nile)
TronChains->>ChainsAbstract: Initialize with chains & CURRENCY.ETH
ChainsAbstract->>ChainsAbstract: Register chain names
ChainsAbstract->>ChainsAbstract: Attempt to add native currencies
Note over ChainsAbstract: No native TRX currencies defined,<br/>so none are added
App->>TronChains: isChainSupported('tron')
TronChains-->>App: true
App->>TronChains: getChainId('tron')
TronChains->>TronDef: Access tron.chainId
TronDef-->>TronChains: 'tron'
TronChains-->>App: 'tron'
App->>TronChains: Access currencies for tron
TronChains->>TronDef: Access tron.currencies
TronDef-->>App: {USDT, USDC definitions}
|
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.
No files reviewed, no comments
6567549 to
5897325
Compare
62d10a1 to
2496888
Compare
- Add Tron mainnet and Nile testnet chain configurations - Add TronChains types and exports - Update currency-types with Tron chain support
2496888 to
80b8f33
Compare
MantisClone
left a comment
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.
Looks good to me so far.
Reviewed with Claude Code Opus 4.5

Description of the changes
Added support for Tron blockchain and its Nile testnet:
Closes RequestNetwork/private-issues#223
Closes RequestNetwork/private-issues#224