Skip to content

Docs for Communities#1526

Merged
thibaultamartin merged 5 commits into
zolafrom
zola-docs-communities
Nov 18, 2022
Merged

Docs for Communities#1526
thibaultamartin merged 5 commits into
zolafrom
zola-docs-communities

Conversation

@thibaultamartin

Copy link
Copy Markdown
Contributor

No description provided.

@thibaultamartin
thibaultamartin marked this pull request as draft October 25, 2022 07:58
This was referenced Nov 1, 2022
@thibaultamartin
thibaultamartin marked this pull request as ready for review November 16, 2022 15:54

@jplatte jplatte left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Lots of small wording suggestions but I don't think this needs a second round of review so I'll approve now.

Comment thread content/docs/communities/_index.md Outdated
Comment thread content/docs/communities/_index.md Outdated
Comment thread content/docs/communities/_index.md Outdated
![](./from-discord.png)

If you're interested in self-hosting your bridge or discovering all the Discord
bridge implementations, please head to the [Ecosystem > Bridges](/ecosystem/bridges)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I really don't like angle brackets in text. should look better (and this one is available on any regular system).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I didn't use those for now because we may want to discuss it further. My major concern is that this character is not easily available on keyboards, I'd rather find a solution for which we have better consistency guarantees.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It's available as ›, does that help? 😅

Comment on lines +26 to +27
- **Users** can send messages, media, reactions and redact their own messages in
a room.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is redacting your own messages something that can be configured away? This makes it sounds like that a bit.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's not, but I wanted to give an overview of the user abilities. Do you have better phrasing in mind?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could be a separate statement righr before or after the listing that everybody can redact their own messages?

Comment thread content/docs/communities/moderation/_index.md Outdated
Comment thread content/docs/communities/moderation/_index.md Outdated

And that's really what mjolnir is: a security guard for your rooms. Because of
the very decentralised nature of Matrix, you need to add the robot to guard the
door of each of your rooms, and you need to grant it the Administrator role so

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is kinda bad, really (because you can't demote it again)..

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That's a very good point. In practice it depends:

  • If mjolnir runs with an account you created, you can use the credentials to demote it
  • If mjolnir runs with an account from a homeserver you have admin access to, you can use the admin API to demote it
  • If you run a mjolnir instance with an account you don't have the credentials of, from a homeserver you don't have (indirect) admin access to, then indeed you can't demote it. This is going to be an issue when mjolnir for all is released for example.

The alternative is to update every room to lower several of the permissions from 100 to e.g. 99. It's quite cumbersome and error-prone. Worth discussing internally

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

IIRC for Ruma I lowered a bunch of things to 50, but maybe it's not a good idea in general 😅

Comment thread content/docs/communities/moderation/_index.md
Comment thread content/docs/communities/switching-providers/_index.md
@thibaultamartin
thibaultamartin merged commit 63178bc into zola Nov 18, 2022
@thibaultamartin
thibaultamartin deleted the zola-docs-communities branch November 18, 2022 10:43
thibaultamartin added a commit that referenced this pull request Dec 2, 2022
* Add docs for communities
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.

2 participants