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

A proposed recommendation for using 'e' and 'p' tags in nostr clients. #1

Merged
merged 4 commits into from
May 4, 2022
Merged

Conversation

unclebob
Copy link
Contributor

@unclebob unclebob commented May 4, 2022

This appears to conform to the conventions used in branle, damus, and more-speech.

Copy link
Collaborator

@eskema eskema left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what happens when it's not a reply but there's a mentioned event? would't that conflict and look like a direct reply?

@fiatjaf
Copy link
Member

fiatjaf commented May 4, 2022

In my opinion "mentions" should be rendered as replies too, but since other clients may not like that it's probably fine to keep that out of this set of recommendations.

Let's make this NIP-10.

@unclebob
Copy link
Contributor Author

unclebob commented May 4, 2022

NIP-13

@unclebob
Copy link
Contributor Author

unclebob commented May 4, 2022

Renamed to NIP-10.

@fiatjaf
Copy link
Member

fiatjaf commented May 4, 2022

Could have been 13 too, I'm just assigning numbers based on how I feel them. But 10 it is!

@fiatjaf fiatjaf merged commit b616aef into nostr-protocol:master May 4, 2022
@fiatjaf
Copy link
Member

fiatjaf commented May 4, 2022

@unclebob you think instead of writing optional on the other NIPs we should always do mandatory?

jb55 pushed a commit that referenced this pull request Aug 15, 2022
monlovesmango added a commit to monlovesmango/nips that referenced this pull request Nov 28, 2022
two parts to this:
1. explicitly calling out 'delegation' as an exception to only single letter tags being queryable
2. adding use case of 'l' being used to identify language

I think nostr-protocol#1 should definitely be added. happy to drop nostr-protocol#2 if people disagree.
@monlovesmango monlovesmango mentioned this pull request Feb 17, 2023
@staab staab mentioned this pull request Mar 28, 2023
@arthurfranca arthurfranca mentioned this pull request Nov 14, 2023
conduition pushed a commit to conduition/nips that referenced this pull request Aug 30, 2024
additional clarity around message content and tags
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.

3 participants