Skip to content

NIP-72: Moderated communities #602

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.

Sign up for GitHub

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

Merged
fiatjaf merged 19 commits into from
Aug 8, 2023
Merged

NIP-72: Moderated communities #602

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
81359b1
Simple moderated communities flow.
vitorpamplona Jun 12, 2023
a933e03
Typos
vitorpamplona Jun 12, 2023
22c1af8
Fixing indentation
vitorpamplona Jun 12, 2023
409fc87
Adds preferred relay information to tags.
vitorpamplona Jun 13, 2023
99b2796
Update 172.md
vitorpamplona Jun 13, 2023
1982df7
Removes quotes from kind
vitorpamplona Jun 15, 2023
9b22109
Fix typos in community image
vitorpamplona Jun 15, 2023
7e0f2fd
Fix formatting
vitorpamplona Jun 15, 2023
0392e21
Fixes double-quoted kind number
vitorpamplona Jun 15, 2023
e4273cc
Moves post approval to a regular (non-replaceable event)
vitorpamplona Jun 15, 2023
f97be25
Adds arthurfranca as author
vitorpamplona Jun 15, 2023
3ef96de
Fixes typos & clarifies the text.
vitorpamplona Jun 15, 2023
b3b9d85
Adds references to other NIPs with more information and standardizes …
vitorpamplona Jun 15, 2023
5e3a7fe
Addressing how to unapprove a post.
vitorpamplona Jun 15, 2023
f7708e4
Rewriting to use specification verbs SHOULD, MAY, etc
vitorpamplona Jun 15, 2023
e5c0603
Better describes the use of `a`, `e` and `p` tags in the post approva…
vitorpamplona Jun 15, 2023
2eceac9
Allows replaceable events to be part of communities as well.
vitorpamplona Jun 15, 2023
c83cf00
Adds relay markers
vitorpamplona Jun 19, 2023
3c4963b
added a third option to approve replaceable events.
vitorpamplona Jul 7, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fixing indentation
  • Loading branch information
@vitorpamplona
vitorpamplona committed Jun 12, 2023
commit 22c1af86227ea09c15fbcc6d34bf14711847e4e4
10 changes: 5 additions & 5 deletions 172.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ Kind 34550 should include any field that helps define the community and the set
"kind": "34550",
"tags": [
["d", "<community_name>"],
["description", "<community_description>"],
["description", "<community_description>"],

// moderators
["p", "<32-bytes hex of a pubkey>", "<optional recommended relay URL>", "moderator"]
["p", "<32-bytes hex of a pubkey>", "<optional recommended relay URL>", "moderator"]
["p", "<32-bytes hex of a pubkey>", "<optional recommended relay URL>", "moderator"]
["p", "<32-bytes hex of a pubkey>", "<optional recommended relay URL>", "moderator"]
["p", "<32-bytes hex of a pubkey>", "<optional recommended relay URL>", "moderator"]
]
}
```
Expand Down Expand Up @@ -61,8 +61,8 @@ The post-approval event includes a stringified `new post request` event inside t
"kind": "34551",
Copy link
Contributor

@arthurfranca arthurfranca Jun 15, 2023

Choose a reason for hiding this comment

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

This parameterized replaceable event is missing a "d" tag. Considering the user can be a moderator on many communities and may approve the same event id for more than one group, i think it would be this giant string: ["d", "34550:<community event author pubkey>:<d-identifier of the community>:<Post Request ID>"].

Maybe it is better as a regular event in the 1000 <= n < 10000 range like 4550 or 4551 with no "d" tag. I don't know.

Copy link
Collaborator Author

@vitorpamplona vitorpamplona Jun 15, 2023

Choose a reason for hiding this comment

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

Good point. I think we can move it to a regular event with event deletion to revert things.

"tags": [
["a", "34550:<community event author pubkey>:<d-identifier of the community>", "<optional relay url>"],
["e", "<Post Request ID>", "<optional relay url>"],
["p", "<Post Request Author ID>", "<optional relay url>"],
["e", "<Post Request ID>", "<optional relay url>"],
["p", "<Post Request Author ID>", "<optional relay url>"],
],
Copy link
Contributor

@arthurfranca arthurfranca Jul 4, 2023

Choose a reason for hiding this comment

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

Suggested change
],
["!", "<Community event author pubkey>"],
],

This would use #539 to make the community owner able to delete post approvals created by others (moderators).

Currently the owner can only delete its own approvals or remove moderators (which deletes all approvals from said moderator).

"content": "{ <New Post Request JSON> }"
}
Expand Down