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

Replace NIP93 Profile Gallery with Nip68 Gallery #1208

Closed

Conversation

believethehype
Copy link
Contributor

@believethehype believethehype commented Dec 3, 2024

Rationale:

  • NIP93 didn't get merged, no other client uses it.
  • Nip68 (Kind 20 events) are considered "High quality posts" in Olas and they land in the user's gallery in Olas anyways.
  • This gives us a way to have syncronized galleries over multiple clients, NIP93 then is obsolete.
  • Instead of clicking an image and hitting "add to gallery" with referencing to the original note, this simply references to the the kind 20 event. Kind 20 events can be posted in the multimedia (aka tt/insta) feed, and eventually reposted/quote posted to the regular kind 1 feed.

This is an alternative PR to #1210

  • Additionally, fixes kind20 description not being added to "content" field.

Rationale:

- NIP93 didn't get merged, no other client uses it.
- Nip68 (Kind 20 events) are considered "High quality posts" in Olas and they land in the user's gallery in Olas anyways.
- This gives us a way to have syncronized galleries over multiple clients, NIP93 then is obsolete.
- Instead of clicking an image and hitting "add to gallery" with referencing to the original note, this simply references to the the kind 20 event.  Kind 20 events can be posted in the multimedia (aka tt/insta) feed, and eventually reposted/quote posted to the regular kind 1 feed.

- Additionally, fixes kind20 description not being added to "title" field.
(can't click videos as of now)
@vitorpamplona
Copy link
Owner

Went with #1210

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