Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement NIP-47 Wallet Connect #200

Open
radmakr opened this issue Nov 22, 2024 · 1 comment
Open

Implement NIP-47 Wallet Connect #200

radmakr opened this issue Nov 22, 2024 · 1 comment

Comments

@radmakr
Copy link

radmakr commented Nov 22, 2024

Nip 47

I am starting this work in my fork. Unless someone else already has it implemented in an un-merged branch?

@tyiu anything I need to know before contributing to this project?
Any tips for implementing a new NIP?

@tyiu
Copy link
Contributor

tyiu commented Nov 23, 2024

That's great! Thanks for contributing to the project. No one has done any work on NIP-47. Please go ahead.

We try to make components of event kinds as composable and reusable as possible. For example, there are protocol extensions with the suffix Interpreting for some common tags. Not sure if any apply here. I also see that parts of the NIP might involve NIP-04 encryption/decryption, and that already exists.

If this NIP requires the client to create any event kinds, know that I'm trying to deprecate the EventCreating protocol in favor of NostrEvent.Builder as it's more flexible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants