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

NIP-09 Deletion Event Improvements #1141

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

vitorpamplona
Copy link
Collaborator

@vitorpamplona vitorpamplona commented Mar 28, 2024

  1. Simplifies the wording
  2. Removes duplicated text
  3. Removes behavior ideas for clients and relays.
  4. Adds k-tag support
  5. Adds a-tag's time limitation (deletes everything up to the event's created at).

Read here

Supercedes @basantagoswami's PR #1069

Fixes: #779

@vitorpamplona vitorpamplona changed the title NIP-09 Deletion Event NIP-09 Deletion Event Improvements Mar 28, 2024
@vitorpamplona
Copy link
Collaborator Author

Also, I was today years old when I realized that the old text allowed a content edit: "The content field MAY also be used to replace the deleted events' own content, although a user interface should clearly indicate that this is a deletion reason, not the original content."

I got rid of this since no one is using Delete events to replace content.

09.md Outdated Show resolved Hide resolved
09.md Outdated Show resolved Hide resolved
@vitorpamplona
Copy link
Collaborator Author

Can we merge this? @fiatjaf

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.

Does a Deletion Event with an a-Tag bans the d-Tag forever?
2 participants