Skip to content

Adds Open Graph "iMeta" tags#1674

Open
vitorpamplona wants to merge 1 commit into
nostr-protocol:masterfrom
vitorpamplona:iopengraph
Open

Adds Open Graph "iMeta" tags#1674
vitorpamplona wants to merge 1 commit into
nostr-protocol:masterfrom
vitorpamplona:iopengraph

Conversation

@vitorpamplona

Copy link
Copy Markdown
Collaborator

This helps clients preview URLs without pinging them.

Read here

@alexgleason

Copy link
Copy Markdown
Member

Why not extend r tags?

@vitorpamplona

vitorpamplona commented Jan 5, 2025

Copy link
Copy Markdown
Collaborator Author

Mostly because of the positional vs variadic nature of the proposal here. It would be basically a copy of the iMeta code. I don't think it is a good idea to move from ["r", "<URL>"] to ["r", "url <URL>"] just to match the variatic component.

Do we want to mix the two? Like on ["r", "<URL>", "<variadic name1> <variadic value1>", "<variadic name2> <variadic value2>", ...]?

I don't know...

@staab

staab commented Jan 6, 2025

Copy link
Copy Markdown
Member

I like this

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