Skip to content

Update 51.md to add support for video lists#972

Merged
pablof7z merged 2 commits into
nostr-protocol:masterfrom
zmeyer44:patch-3
Jan 8, 2024
Merged

Update 51.md to add support for video lists#972
pablof7z merged 2 commits into
nostr-protocol:masterfrom
zmeyer44:patch-3

Conversation

@zmeyer44
Copy link
Copy Markdown
Contributor

@zmeyer44 zmeyer44 commented Jan 7, 2024

The thought is to enable users to create playlists or better categorized lists of videos

@zmeyer44 zmeyer44 marked this pull request as ready for review January 7, 2024 15:38
@pablof7z
Copy link
Copy Markdown
Member

pablof7z commented Jan 7, 2024

Why kind:1 notes?

@zmeyer44
Copy link
Copy Markdown
Contributor Author

zmeyer44 commented Jan 7, 2024

Why kind:1 notes?

I just did it bc kind 30004 lists had them. But probably not necessary. I'll remove

@pablof7z
Copy link
Copy Markdown
Member

pablof7z commented Jan 7, 2024

I think for 30023 curations it makes sense: a curation of long and short notes.but a curation of videos and short notes is weird imo

@pablof7z pablof7z merged commit e489ed4 into nostr-protocol:master Jan 8, 2024
@AsaiToshiya
Copy link
Copy Markdown
Collaborator

Doesn't it support vertical videos (kind: 34236)?

Just out of curiosity, why was this PR merged? Video Events is not merged yet.

pats2sats pushed a commit to sudonym-btc/accommodation-nip that referenced this pull request May 20, 2026
Update 51.md to add support for video lists
francismars pushed a commit to francismars/nips that referenced this pull request May 24, 2026
Update 51.md to add support for video lists
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