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-28: Public Chat #38

Merged
merged 31 commits into from
Sep 10, 2022
Merged

NIP-28: Public Chat #38

merged 31 commits into from
Sep 10, 2022

Conversation

AtlantisPleb
Copy link
Contributor

@AtlantisPleb AtlantisPleb commented Aug 24, 2022

Let's also disrupt Telegram, FB Messenger, WhatsApp, WeChat, et al.

Link to NIP-28

@fiatjaf
Copy link
Member

fiatjaf commented Aug 24, 2022

I would say bring back channel metadata to kind 41 since it doesn't fit the concept of replaceable event as defined by NIP-16 anyway. Relays will just have to add a custom handler for kind 41 events just like they do already for kind 0 and kind 3.

Aside from that all looks great.


The `content` may optionally include metadata such as a `reason`.

Clients SHOULD hide event 42s shown to a given user, if there is an event 43 from that user matching the event 42 `id`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this could work for kind1 messages as well, it's something I've wanted for hiding replies to posts in damus.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This could be done by a special reaction, like '!' or something like that, which if emitted should signify that the comment replied to should not be shown to the given user. Its a sort of 'hide' indicator.


The `content` may optionally include metadata such as a `reason`.

Clients SHOULD hide event 42s shown to a given user, if there is an event 44 from that user matching the event 42 `pubkey`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same for this one

@fiatjaf fiatjaf merged commit 3423a6d into nostr-protocol:master Sep 10, 2022
@fiatjaf
Copy link
Member

fiatjaf commented Sep 10, 2022

Can someone update the README with the new kinds and a link to the NIP?

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.

6 participants