Remove external payment options from payment request modal about bitcoin-connect HOT 4 CLOSED

Hi @positiveblue

Optimally Polling the server to check if the payment has been received would be the best option, but I can see it might not always be possible.

Maybe we can have a allowExternalPayment boolean that is default to true. If false, it hides the external payment options.

CC @reneaaron @bumi

from bitcoin-connect.

should we consider yet another section with a text field (from) for introducing the preimage manually

No, I think users should never have to manually paste a preimage. Most people have no idea what it is, and I'm not sure what they gain from doing this or even where they get that preimage from

from bitcoin-connect.

It also makes sense to only show the external options in some cases. I wonder if instead we add a property paymentMethods which can be "all" or "internal" or "external"

from bitcoin-connect.

Polling the server to check if the payment has been received would be the best option

Agree. We take the preimage as proof of payment so it's possible that the server/service that takes the credentials does not have access to an invoice status.

Maybe we can have a allowExternalPayment boolean that is default to true.

I agree that by default most of the people want it set to true. So whatever route we follow (showEncodedInvocie or hideEncodedInvoice the default should to have the QR visible.

Also, should we consider yet another section with a text field (from) for introducing the preimage manually (I would go with no, but in case you see it necessary)

from bitcoin-connect.