multicoin-address-validator
multicoin-address-validator copied to clipboard
Fix AVAX, include AVAXC
The current AVAX implementation is actually for the C-Chain, which is actually AVAXC according to SLIP-0044.
These changes are breaking but accomplish two things:
- Fix the AVAX implementation to parse X- & P- chains instead of the C-Chain
- Add in AVAXC support to continue parsing the C-Chain
A couple questions I have:
- Would you like the
dist/
folder committed as well? - Would you prefer the ID stripping to be its own function? My thinking was that it was unnecessary abstraction to make it a function
References:
The error in Travis looks like a problem with Node 18 and Travis, not the changes here.
Sorry to ping you @christsim, is there any chance I can get a review on this?