Skip to content

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

Merged
merged 1 commit into from
Oct 18, 2016
Merged

Conversation

mweinelt
Copy link
Contributor

Documentation for the Matrix platform submitted in home-assistant/home-assistant#3827.

home-assistant/core#3827

@robbiet480 robbiet480 added the Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! label Oct 12, 2016
Copy link
Contributor

@Landrash Landrash left a 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 @@
--
Copy link
Contributor

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.
Copy link
Contributor

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
Copy link
Contributor

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.
Copy link
Contributor

@Landrash Landrash Oct 12, 2016

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.

@mweinelt
Copy link
Contributor Author

Ok, applied the requested changes.

Copy link
Member

@fabaff fabaff left a 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 🐦

@fabaff fabaff merged commit 3aae508 into home-assistant:next Oct 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants