implement nip57 with suggested modifications

[fiatjaf]

This implements modifications suggested on nostr-protocol/nips#224 (comment) while retaining backwards-compatibility (although I think it should not retain that).

I wasn't able to test it yet because I don't have a compatible server (I was turning https://github.com/nbd-wtf/satdress into a zap-compatible server but realized that that will be very hard because currently it cannot know when an invoice was paid), so maybe it is broken.

For now, I'm leaving here just as an exercise.

[v0l]

I implemented the new zaps flow api already in snort donate, ill push it later to test

[v0l]

Ok its live now, you can try use the new zaps flow here: https://api.snort.social/swagger/index.html

Payment status is only checked in background job so it will take about 2mins to see receipt

[fiatjaf]

Thank you. I can only try this tomorrow, will proceed from there.

[v0l]

Thank you. I can only try this tomorrow, will proceed from there.

I already tried it out, i implemented it differently on the API

POST /zap returns something similar to LNURL endpoint:

{ "pr": "lnbc1..." }

I also implement the same status response as LNURL

[fiatjaf]

You mean the status: "ERROR" and the reason string? OK, that sounds good to me.

[v0l]

Yep, pushed the changes, I got as far as making zaps but they are not showing in the UI