Add tron namespace and CAIP-2 specification#162
Add tron namespace and CAIP-2 specification#162danroc wants to merge 4 commits intoChainAgnostic:mainfrom
tron namespace and CAIP-2 specification#162Conversation
abf759e to
32f105a
Compare
tron namespace and CAIP-2 specification
|
Made minor comments for formatting and made obvious stuff painfully explicit (it's been my experience that these get read very quickly!). One editorial aside that might be worth mentioning is that proper handling of large ChainIDs is a little hard to guarantee/assume. I remember there have been many discussions over the years (mostly linked from this eth-mag thread ) about what the proper or optimal maximum value should be for chainId, but I also remember (can't find the link right now) that viem and other major libraries validate chainIds internally, making a defacto maximum value of... whatever library a given dapp uses 😅 This is all probably overkill for a TRON namespace, but just a possible footnote if you're already/also tracking that thread elsewhere... |
|
@obstropolos care to double-check i'm not missing anything? |
|
Thanks, @bumblefudge. I’m not very familiar with the discussion you mentioned, would you like to write the editorial note? Otherwise, we can merge the current version. |
This PR adds the
tronnamespace and its CAIP-2 specification.Rendered files: