escapedcat comments

Results 637 comments of


                                            escapedcat

Use minSendable and maxSendable values from LNURLp response to cap user input

Awesome @fanismichalakis ! Thanks for tackling this! Let us know if you get stuck or need any feedback.

Github bug/feature templates: Switch to forms

Thanks @buneeIsSlo ! Assigned you. Let us know if you need any further info or feedback.

Github bug/feature templates: Switch to forms

> Hi! @escapedcat I've raised a PR, could you please take a look at it? Saw it, thinking about how to test it. Eventually we're just gonna merge it and...

Github bug/feature templates: Switch to forms

Thanks for you PR! I commented.

Send satoshis window should have the back button too

Note: We use this screen in also in the prompt. There we do not need a "back" button.

Sometimes fiat value appear as 0 but should show none info or other kind of rendering

@bitcoinuser is the `0` usually replaced by a fiat value after some time/seconds or does it stay? If this just shows during loading we could i.e. just show `...` if...

Sometimes fiat value appear as 0 but should show none info or other kind of rendering

Hm, currency depends on the settings. So checking for i.e. `$` will not cover all cases, right? Play around with the currencies in "settings". Wanna check what solution might work...

feat: use resultcard in lnurlchannel

![image](https://user-images.githubusercontent.com/1016218/194802673-ddf96288-bc61-475c-b809-d2925537108d.png) Should we take care of that `undefined` in the response? Like say "No nodeId available" or just leave it out if it's undefined?

feat: use resultcard in lnurlchannel

lgtm, we see the k1 and the node-id like before ![image](https://user-images.githubusercontent.com/1016218/198045902-517a5ea5-2965-4df0-bc1f-93266e977008.png)

feat: use resultcard in lnurlchannel

I assume bumi wanted to merge