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

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
Sources/NostrSDK/EventKind.swift Outdated Show resolved Hide resolved
demo/NostrSDKDemo/Demo Views/QueryRelayDemoView.swift Outdated Show resolved Hide resolved
@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
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

Successfully merging this pull request may close these issues.

2 participants