nips icon indicating copy to clipboard operation
nips copied to clipboard

NIP-10: Add marked p tags

Open Semisol opened this issue 2 years ago • 6 comments
trafficstars

Semisol avatar Feb 23 '23 21:02 Semisol

Is this implemented somewhere? Was it discussed?

fiatjaf avatar Feb 23 '23 21:02 fiatjaf

I was for marked p tags before, but I'm against now. they achieve nothing and promote sharing misinformation. if someone is to not be tagged for notifications, then the tag should be removed. clients up to date should have a simple way to show the user who is beeing tagged and given the option to remove unwanted ones. this change will simply introduce complexity with questionable benefits, unless I'm not seeing something...

eskema avatar Feb 24 '23 01:02 eskema

clients up to date should have a simple way to show the user who is beeing tagged and given the option to remove unwanted ones.

Gossip had that once. I had to remove it for some temporary reason I don't even remember.

mikedilger avatar Feb 24 '23 01:02 mikedilger

Copy any p tags from the event being replied to, if any...

I know this is somewhat already written before this PR, but do we really want to notify all p chain when many levels deep in the tree? (Also it makes event heavier) For instance, does the root user want to be notified of a reply not directed to him, just because it is part of the thread he started?

silent: This public key is being referenced in this note but should not be notified (such as public key lists).

Don't the mentioned pubkey deserve to be notified when not a private message? Sorry I didn't understand the "public key lists" example.

arthurfranca avatar Feb 24 '23 18:02 arthurfranca

I think it would be better to just exclude the p tag and to simply put the pubkey in the content, then it would be up to clients to extract this pubkey as a mention which is clickable.

Now you have "mentioned" without notifying them.

v0l avatar Mar 01 '23 13:03 v0l

You could also use a different tag, like ["pp", "pubkey"] instead of p, so it won't show up as a notification. It has the benefit of not introducing yet another thing for clients to parse (although some clients will require a micro update).

fiatjaf avatar Mar 01 '23 15:03 fiatjaf