-
Notifications
You must be signed in to change notification settings - Fork 149
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
Feature Request: btcd node #302
Comments
Yes, adding support for btcd was an original goal of mine since the beginning. It's been lower priority for me because I felt it important to focus on the features that were specifically requested by users first. It would be great if you wanted to work on this. I have no objections. I have used btcd for my simnet lightning nodes in the past but have never written an app utilizing its RPC interface, so I do not know if there are any good npm packages available like the one we use for bitcoind. Based on my experience adding c-lightning support, I would assume that this feature will likely require some refactoring of the UI components, redux models, and services. I tried to architect the app to support multiple implementations from the start but I know it's not 100% there today. Feel free to take a shot at implementing this and let me know if you have any questions. Thanks for your help 👍 |
Hello @jamaljsr I would like to try my hand on this. But a little question: Other than writing the dockerfile for btcd, Is there a particular file in src folder I can look at because I noticed by default polar checks for the bitcoin core image over the fly |
Hi @NonsoAmadi10, thanks for offering to give this a go. I think this will require a pretty big update to the codebase, but if you're willing to give it a shot, here's a high level list of things would need to be implemented:
I think that covers most of what needs to be done. Obviously, the devil is in the details on how to get all of this working. Feel free to ask any more questions. |
hey i also looking forward to thie feature regards |
Hi @jamaljsr I am back again lol. I tried my hands on this based on the information but as you know the devil is in the details. I hit a road block:
You can see the work I have done so far here. I know you are very busy but I feel this PR is almost ready but I am missing one fine details. Thanks in advance |
Hey @NonsoAmadi10 thanks for the updates. You do not need to push to docker hub in order to test. Just build the image and it will be available locally on your machine to use.
|
Hey @jamaljsr, I've decided to take a stab at this. I noticed that @NonsoAmadi10 has started something, so I'll go through it and pick it up from there. |
Hey @Abdulkbk |
Hey @kelvinator07 and @Abdulkbk this looks interesting 🤔. Let me know if you need an extra hand on this? and I'd love to help in any way I can. |
I saw that this was included in the TODO, but I would like to open an issue about it so we can discuss and keep track of it.
I would like to be able to use btcd as a backend instead of bitcoind.
As lnd makes use of Neutrino SPV for mobile and embedded devices, it's currently not possible to have a complete test environment using Polar as there's no support for the Neutrino protocol in bitcoind yet.
I could work on this if it hasn't already been started, but I cannot tell when it will be done.
Cheers
Hampus
The text was updated successfully, but these errors were encountered: