-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Document the matrix notification platform #1215
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some small formatting changes are needed and a question about maybe clarification.
@@ -0,0 +1,40 @@ | |||
-- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing a -
--- | ||
|
||
|
||
The `matrix` platform allows you to deliver notifications from Home Assistant to a matrix room. Rooms can be both direct as well as group chats. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Matrix without capital letter.
```yaml | ||
# Example configuration.yaml entry | ||
notify: | ||
- name: NOTIFIER_NAME |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove the notifier name variable. In general we try to minimize the examples since they are often cut and pasted.
- **room** (*Required*): The room the messages will be sent to. | ||
- **verify_ssl** (*Optional*): Verify the homeservers certificate. Defaults to `true`. | ||
|
||
The target room has to be precreated, the room id can be obtained from the rooms settings dialog. Rooms by default have a canonical id of the form `!<randomid>:homeserver.tld`, but can also be allocated aliases like `#roomname:homeserver.tld`. The notifying account may need to be invited to the room, depending on its policies. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be a good plan to specify that it's the rooms policies that require the invitation.
Ok, applied the requested changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good for me 🐦
Documentation for the
Matrix
platform submitted inhome-assistant/home-assistant#3827
.home-assistant/core#3827