Ride-The-Lightning / RTL

Ride The Lightning - A full function web browser app for LND, C-Lightning and Eclair
MIT License
746 stars 160 forks source link

is refresh required after every UI interaction? #379

Open philbertw4 opened 4 years ago

philbertw4 commented 4 years ago

Describe the bug I have really enjoyed RTL in general. However, in order to use it, I seem to be needing to hit the refresh button in my browser a lot. Is this to be expected?

To Reproduce For example:

  1. Create an invoice
  2. Navigate to another page in RTL after the invoice has been created
  3. Have somebody pay the invoice (confirm via lncli commandline that invoice has been paid)
  4. Navigate to invoices in RTL and check the invoice -- the state of the invoice has not been updated in RTLs display despite the fact that it has been paid.

Your environment

Additional context What I would like to know is whether this is the expected behavior, or whether the UI is supposed to be updating on its own in real time (which, of course, would be much preferred!).

philbertw4 commented 4 years ago

What I'm noticing is similar to #147 which was closed in a much earlier version of RTL. Specifically, when I create an invoice, and am waiting for payment, shouldn't that page be polling/monitoring for the payment? Right now I have to refresh manually.

ShahanaFarooqui commented 4 years ago

@philbertw4 Initially, when we worked on LND invoices, LND did not have invoice/subscribe API and we were unable to subscribe to them. Additionally, this issue is not same as #147. Issue #147 is to update wallet & balance information whenever a payment has been done via the node. It is NOT polling.

@saubyk Please check the invoice/subscribe api and advise.

philbertw4 commented 4 years ago

Ok, thank you for the clarification. In general, I have really enjoyed RTL. You all are doing great work!

3nprob commented 3 years ago

I also get forced to reaauth every ~1-5 minutes - which is super frustrating obviously. There's nothing exceptional in logs and the backend process is uninterrupted. This is for any backend, LND or not.

ShahanaFarooqui commented 3 years ago

@3nprob can you please open a separate issue if you are getting logged out every 5 minutes as this issue is different than that. It will be better to log different problems in separate issues. Also, please update your RTL version if you are still on 0.7.x.

3nprob commented 3 years ago

Got it - was not confident if this was intended behavior or not and it does fit the title (: (I'm on 0.10.0 BTW)

jaonoctus commented 3 years ago

Same issue here. You guys could update the interface or make the requests again and update the data reactively after some user actions.

saubyk commented 3 years ago

@jaonoctus The interface is updated reactive on multiple user actions like channel opening, channel closure, payment execution etc.

But a reactive invoice payment status update cannot be handled as easily. And do note that RTL is a node management interface not a payment solution. Reactive payment status updates are better handled in wallet apps and processors like BTCPayserver.