chore: simplify nip-47#2210
Conversation
- remove multi_methods - reduce duplication regarding encryption and backward compatibility
|
There's still stuff about the nip 04 encrypted events in there. Is multi pay not implemented anywhere? I would think it would be useful for paying zap splits, I would like to see wallets implementing it. |
|
If anyone is interested, they can open a PR with these add-ons, and then we can wait until we have enough support to re-add them. But I do zap splits and don't really use the multi-pay method. |
|
Looks like a reasonable ask. And indeed this system looks too complex. |
There are a lot of apps which still only support NIP-04 I think (and I think a few still only the NIP-04 notification format). So unfortunately we can't remove these yet. |
|
I am gonna merge this for now to simplify further improvements to the NIP (bolt 12). But if any 2+ apps still use these methods (not just coded in the lib, actually being used), feel free to send a new PR with the methods back. |
Following the removal in the spec nostr-protocol/nips#2210
It got removed from the spec so we don't need to support it anymore. nostr-protocol/nips#2210
multi_methods (complex to implement as the flow is quite different, did not gain adoption)CC @vitorpamplona @frnandu @jklein24 @benthecarman
See related PRs / topics: