Skip to content
This repository has been archived by the owner on Mar 15, 2020. It is now read-only.

Sending blog posts should require power #46

Closed
lukebarnard1 opened this issue Apr 1, 2017 · 0 comments
Closed

Sending blog posts should require power #46

lukebarnard1 opened this issue Apr 1, 2017 · 0 comments
Labels
bug Something doesn't work

Comments

@lukebarnard1
Copy link
Owner

I've just realised a fatal flaw in j - anyone can post a blog in a room, even if they're not a moderator. Permission to post messages needs to be handled by the server, not by j (as it is now). This would be a simple matter of putting a power level required to send m.room.messages into room, but comments are also messages.

So comments and blogs need different event types, which can have their own custom power levels individually. Neither should be messages at this point, and messages could be reserved for general discussion about the blog as is intended by in-site messaging: #37.

I'd probably go with

  • j.blog.comment
  • j.blog.post
@lukebarnard1 lukebarnard1 added the bug Something doesn't work label Apr 1, 2017
@lukebarnard1 lukebarnard1 changed the title Sending messages should require power Sending blogs should require power Apr 1, 2017
@lukebarnard1 lukebarnard1 changed the title Sending blogs should require power Sending blog posts should require power Apr 1, 2017
lukebarnard1 added a commit that referenced this issue Oct 29, 2017
This removes compatibility with Riot and other matrix clients supporting m.text using org.matrix.custom.html. The motivation behind this is to introduce different power levels for comments vs. blogs, which can only be done by using different event types. The room can still be used as a general chat room surrounding the blog and this might see some kind of UI in journal in the future.

See #46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something doesn't work
Projects
None yet
Development

No branches or pull requests

1 participant