Add default address encodings for Bitcoin#50
Add default address encodings for Bitcoin#50sidhujag wants to merge 5 commits intocoinbase:masterfrom
Conversation
This should be here so its easily overridable by other UTXO chains.
| ) | ||
|
|
||
| // CreateMainNetParams is a function to override default mainnet settings with address prefixes | ||
| func CreateMainNetParams() (*chaincfg.Params) { |
There was a problem hiding this comment.
@sidhujag would it make more sense to provide a JSON file at initialization that determines the values of chaincfg.Params? If not provided, it would just default to what we already had here.
In the general case, I think this would mean that a fork could integrate without maintaining their own repo? Unless you've seen other things that must be changed?
There was a problem hiding this comment.
Yup that makes sense. Outside of these network changes and the overriding of the default table/log sizes for badger we should be able to just use rosetta-bitcoin. I actually prefer this option. If we can move it out to json then we can configure network settings, the genesis details, the address prefixes and it should just work.
See syscoin#1 for related issue, the CLI also needs configurable setting for the badger log/table defaults. Other than that it is common with the base.
2 participants
This should be here so its easily overridable by other UTXO chains.