Bug: `hold_invoice_accepted` notification has an invalid state

[im-adithya]

Currently the hold_invoice_accepted notification has state set to accepted instead of pending.

Internally we can use accepted but because we use pending in list_transactions too (verified this), it would be best to align both since Nip47Transaction state should be one of "settled" | "pending" | "expired" | "failed"

[im-adithya]

I think the easiest way to do this is to change "accepted" to "pending" in ToNip47Transaction

[krrish-sehgal]

@im-adithya ,Would it be okay if take up this issue?
Thanks!

[rolznz]

@krrish-sehgal sorry this issue was not updated. "accepted" is a valid state and is different to pending. Closing for now.

Also linking nostr-protocol/nips#1913

[krrish-sehgal]

Cool... heading to another!