getAlby / lightning-browser-extension

The Bitcoin Lightning Browser Extension that brings deep Lightning & Nostr integration to the web. Wallet interface to multiple lightning nodes and key signer for Nostr, Liquid and onchain use.
https://getalby.com/#extension
MIT License
540 stars 194 forks source link

[BUG] Handle sub sats invoices correctly #3264

Open reneaaron opened 3 weeks ago

reneaaron commented 3 weeks ago

Is there an existing issue for this?

Describe the bug

https://athenut.com/topup seems to use sub-sat amounts in their invoices:

e.g. lnbc2090820p1pnjp7hrsp52nvfg9qufaw57c74h9n35hwrxmcyn27yf7e9hrcn8ux639yryk0qpp50t2xjm0dpy8lc74e68gannelm6q0yumkau6866usv742md6j2edqdqqxqrpcgcqpjrzjqwryaup9lh50kkranzgcdnn2fgvx390wgj5jd07rwr3vxeje0glc7rf05uqqg8gqqqqqqqlgqqqqrucqjq9qxpqysgqsl4s2r4p3r4f7a2e0p67hvdkmq4ss904q2v3lt8qvmy5arh4jaxnnd8rszpsjxnj0unuweled5vv732dngdhatp8qhwgwccwk8drcxcqay02aq

When you paste this into the extension you will see an amount of 0. image

Also the success screen looks quite odd: image

Screenshots [optional]

No response

Steps To Reproduce

...

Expected behavior

...

Alby information

...

Device information

No response

Additional context

No response

Are you working on this?

None

bumi commented 3 weeks ago

sub sats don't exist