bordalix / helm-wallet

The lightning wallet even your grandma can use
https://helm-wallet.pages.dev
MIT License
18 stars 8 forks source link

Pay lightning invoice without amount from Coinos.io error #8

Closed blankscreengithub closed 2 months ago

blankscreengithub commented 2 months ago

I'm getting a red error at the top of the screen when scanning or pasting an invoice from Coinos.io:

Error decoding invoice

bordalix commented 2 months ago

Can you paste the invoice here? Or decode it with https://lightningdecoder.com/ and see if there are any errors.

blankscreengithub commented 2 months ago

I got it to work by specifying the sats amount in the invoice on Coinos.io.

I get the error when I don't specify the amount in the invoice.

Not sure if this is really a bug, but I can pay Coinos invoices that don't have an amount specified with Strike.

Maybe there could be an option to specify the amount in the Helm Wallet when no amount is specified in the invoice?

bordalix commented 2 months ago

Interesting.

Do you mind generate an invoice without an amount and paste it here?

Your suggestion is easily doable.

bordalix commented 2 months ago

Ahhh, Boltz does not support swaps with invoices without amount, so Helm doesn't to.

You can check by yourself:

Sorry about that.

blankscreengithub commented 2 months ago

Thanks! I kind of figured it was on the boltz side. It's not an issue, but maybe the Helm wallet can pop up with a clearer message that an amount needs to be specified. Grandma is not going to know what "Error decoding invoice" means. ;-)

bordalix commented 2 months ago

Eheheh... There's now a specific error message when that occurs:

"Invoices without amount are not supported"

I think grandma is happy 😛

Thanks for finding this.