geewallet icon indicating copy to clipboard operation
geewallet copied to clipboard

Frontend.Console: ask confirmation from funder

Open parhamsaremi opened this issue 2 years ago • 15 comments

Previously we only asked the funder to press a key to continue the process. But now we ask the funder to state if he wants to continue or not.

parhamsaremi avatar Nov 07 '22 10:11 parhamsaremi

@aarani please review

knocte avatar Nov 14 '22 03:11 knocte

I don't like this error "funder didn't confirm to lock the funding!". geewallet user doesn't know what funder or fundee means.

aarani avatar Nov 14 '22 06:11 aarani

I don't like this error "funder didn't confirm to lock the funding!". geewallet user doesn't know what funder or fundee means.

Do you have any suggestions for the replacement? :thinking:

parhamsaremi avatar Nov 14 '22 09:11 parhamsaremi

Do you have any suggestions for the replacement? 🤔

@aarani ping

knocte avatar Nov 17 '22 01:11 knocte

@parhamsaremi UI messages are mostly @knocte 's area

aarani avatar Nov 17 '22 15:11 aarani

@parhamsaremi UI messages are mostly @knocte 's area

But I pinged you, so I want to know what's your best suggestion first, then I'll improve on that.

knocte avatar Nov 18 '22 03:11 knocte

BTW this PR needs to be rebased (to eliminate the red CI), I believe.

knocte avatar Nov 18 '22 03:11 knocte

But I pinged you, so I want to know what's your best suggestion first, then I'll improve on that.

@parhamsaremi @knocte "Funding is confirmed, awaiting locking" smth like that

aarani avatar Nov 18 '22 07:11 aarani

What does "locking" mean exactly?

knocte avatar Nov 18 '22 08:11 knocte

What does "locking" mean exactly?

Channel Locking, funding_locked.

aarani avatar Nov 21 '22 11:11 aarani

Channel Locking, funding_locked.

What does that mean, for the layman?

knocte avatar Nov 21 '22 11:11 knocte

Channel Locking, funding_locked.

What does that mean, for the layman?

Not sure, we always call it locking, even geewallet ui says locking image

aarani avatar Nov 21 '22 12:11 aarani

Yeah I know we already have some strings that use that term, but I was never convinced about them so I'm taking this opportunity to review this. @aarani how do other wallets call this?

@parhamsaremi why is CI still red?

knocte avatar Nov 22 '22 04:11 knocte

Yeah I know we already have some strings that use that term, but I was never convinced about them so I'm taking this opportunity to review this. @aarani how do other wallets call this?

Unfortunately idk, according to my experience with muun, they completely hide the channel making process.

aarani avatar Nov 22 '22 07:11 aarani

Yeah I know we already have some strings that use that term, but I was never convinced about them so I'm taking this opportunity to review this. @aarani how do other wallets call this?

@parhamsaremi why is CI still red?

Sorry I was focused on other tasks I'll rebase now. Did you reach a final decision about the message?

parhamsaremi avatar Nov 22 '22 08:11 parhamsaremi