C-Otto / lnd-manageJ

MIT License
52 stars 11 forks source link

Feature Request: List pending channels #83

Closed danielbroll closed 1 year ago

danielbroll commented 1 year ago

Is your feature request related to a problem? Please describe. Fees are high, jpgs are clogging up the chain, channel might take a while until they confirm.

Describe the solution you'd like Showing pending channels would help to stay on top! image

Additional context

C-Otto commented 1 year ago

It's tricky, see my comment in your code. What kind of information do you expect to see? For me, lnd-manageJ mostly helps gather information that is tricky to acquire, for example routing statistics or details about on-chain fee costs. For pending channels there's not a whole lot to see, especially if you already know how to run lncli pendingchannels (which, I admit, isn't a GUI).

danielbroll commented 1 year ago

Thanks for taking a look at the branch.

I would like to see:

Would you rather not have those information in the UI to keep the code base lean?

I personally use the UI to get an overview of my node and to see if there is something to do and then use lnci and scripts for the actual execution.

I guess having the feature richness of thunderhub is not the goal for the project? Where to draw the line?

C-Otto commented 1 year ago

The project doesn't really have a goal, I must admit. Personally I don't see the advantage, as I'm quite happy with my current toolset (mostly lncli), and I don't need support from lnd-manageJ for that. On the other hand, I don't use the UI at all, so I'm not really part of the target demographic here. Feel free to implement this, but please don't fiddle to much with the existing code regarding "real" as in non-pending channels.

C-Otto commented 1 year ago

Oh, one more thing. You're only talking about not-yet-opened channels, but if you also consider not-yet-closed channels, this is more interesting. Especially with the recent chaos, getting some answers to questions like the following would help. I'm not too happy with my (bash-script based) toolchain, so integrating this into lnd-manageJ might help.

danielbroll commented 1 year ago

I create a separate issue for your feature request https://github.com/C-Otto/lnd-manageJ/issues/85

C-Otto commented 1 year ago

Fixed by #84