Skip to content
New issue

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

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

Already on GitHub? # to your account

Change NIP-04 direct message and encryption naming to avoid ambiguity with NIP-17 #158

Merged
merged 5 commits into from
May 28, 2024

Conversation

tyiu
Copy link
Contributor

@tyiu tyiu commented May 21, 2024

These two NIPs overload the term "direct message", so we need to use the term "legacy" for the old one to differentiate them.
NIP-04 - Encrypted Direct Messages
NIP-17 - Private Direct Messages

Note that this PR contains breaking changes.

@tyiu tyiu force-pushed the tyiu/change-nip04-naming branch from b62716c to 32325aa Compare May 21, 2024 04:08
@tyiu tyiu requested review from joelklabo and bryanmontz May 27, 2024 03:03
@tyiu tyiu requested a review from bryanmontz May 28, 2024 05:06
@tyiu tyiu merged commit 78aeb3b into main May 28, 2024
4 checks passed
@tyiu tyiu deleted the tyiu/change-nip04-naming branch May 28, 2024 17:54
RandyMcMillan pushed a commit to RandyMcMillan/nostr-sdk-ios that referenced this pull request Sep 1, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants