-
Notifications
You must be signed in to change notification settings - Fork 146
Add LBC asset #3358
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: master
Are you sure you want to change the base?
Add LBC asset #3358
Conversation
|
Looks like a good start. Did you also look at client code in |
|
Nope. |
|
The regnet_test can be run: go test -v -count=1 -tag=harness -run <your test name>Are you using doge as a pattern? .. maybe look at |
|
I honestly don't know. I thought just adding a file with some LBC specific parameters would be enough. I didn't plan to write multiple functions in a codebase I'm not familiar with. |
|
All the coins except base have a simnet we can spin up in dcrdex/dex/testing. It would be good to have one for this too as it makes testing a billion times easier. Does lbry requires nodes other than the main daemon? Like eth that requires separate stake nodes? |
|
I guess not. It is just a Bitcoin fork with some different chainparams. |
|
Should just require an extra script like ltc or doge and a README then |
could you please explain what those "harness" and "simnet" are ? we ll try to comply.any docs around to read ? |
|
As far as I know, Simnet is a local running blockchain network to SIMulate transactions, where as Mainnet and Testnet are publicly accessible networks. |
Some of the coin harnesses do have a README.md with dev notes. Example: BTC, DCR, FIRO, LTC, readme's. Firo
Coin Harnesses:
cd ~/dex/dcrdex/client/asset/firo
go test -v -count=1 -tags=harness -run TestWallet
Note: these scripts call the CLI RPC tools like Example: Server Harness:
Does your current code work on your mainnet/testnet? |
|
Look at idk if you want to add everything in this pr, but this is also missing changes to the web ui |
|
I'm not planning to get familiar with this codebase. The maintainers can ask me information about LBC and the blockchain, but I doubt this PR will get merged. Maybe I even should close it and make an issue instead. |
|
Does your current code work on your mainnet/testnet? |
|
No idea. |
|
There is an issue #3352 It's your call. We were hoping to just review and yes merge when we are able to test it. |
Know how you feel .. felt same way when I first did Firo 2 years back. But then it started to work! IDE vscode
Build (linux) debug version of
|
This pull request will add the LBC asset.