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-85: d Tag References in Addressable Events #1536

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alexgleason
Copy link
Member

This is an alternative solution to #1510 inspired by the discussion in #1534

@vitorpamplona
Copy link
Collaborator

Can you clarify again what is this fixing? I went back into the old discussions but it's hard to find the actual problem.

@alexgleason
Copy link
Member Author

@vitorpamplona Relay hints. Putting them on the d tag breaks conventions. People already have code that does things with "p" and "e" tags automatically, since they're considered "standardized tags". Querying with nak for example has special CLI flags for #p queries. Another example is using p-tags to connect nodes in a graph database. Semantically, it just should have them. This method ties them together. But it's only useful if people commit to doing it strictly so it can always be depended upon.

@vitorpamplona
Copy link
Collaborator

ohhh interesting..

Ack

@staab
Copy link
Member

staab commented Oct 14, 2024

In pretty much every case I've seen it's better for d tags to be random. Maybe p/e tags would make sense though, since they're immutable, but what's the use case?

@alexgleason
Copy link
Member Author

@staab In Ditto I need to manage records about other users and events, about things that don't make sense to use labels.

#1534 is a good example. Also: https://docs.soapbox.pub/ditto/nsec#user-records

This probably doesn't deserve to be a NIP on its own, but #1510 exists so I went for it.

@arthurfranca
Copy link
Contributor

I think #1540 could be a better alternative

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.

4 participants