-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Improve Z-Wave intro getting started info and readability #40562
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
base: current
Are you sure you want to change the base?
Conversation
Update Z-Wave integration introduction to match the into from the ZHA integration.
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
misspelling, changed refered to referred
- [Binary sensor](/integrations/binary_sensor/) | ||
- [Button](/integrations/button/) | ||
- [Climate](/integrations/climate/) | ||
- [Cover](/integrations/cover/) | ||
- [Diagnostics](/integrations/event/) | ||
- [Event](/integrations/diagnostics/) | ||
- [Fan](/integrations/fan/) | ||
- [Humidifier](/integrations/humidifier/) | ||
- [Light](/integrations/light/) | ||
- [Lock](/integrations/lock/) | ||
- [Number](/integrations/number/) (i.e. analog input/output) | ||
- [Select](/integrations/select/) | ||
- [Sensor](/integrations/sensor/) | ||
- [Siren](/integrations/siren/) | ||
- [Switch](/integrations/switch/) | ||
- [Update](/integrations/update/) |
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.
We should not add TOCs here. We have those in the sidebar.
@@ -51,15 +51,36 @@ related: | |||
title: Other Z-Wave adapters | |||
--- | |||
|
|||
The **Z-Wave** {% term integration %} allows you to control a Z-Wave network from Home Assistant via the [Z-Wave JS](https://zwave-js.github.io/node-zwave-js/#/) driver. | |||
Z-Wave JS is a hardware-independent [Z-Wave controller](https://z-wavealliance.org/) implementation that can replace most proprietary Z-Wave controllers (also sometimes referred to as a Z-Wave hub, Z-Wave gateway, or Z-Wave bridge). |
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.
This is not the Z-Wave JS integration, the first wording change is thus incorrect.
implementation that can replace most proprietary Z-Wave controllers
Sounds irrelevant.
Having this as a first paragraph doesn't tell or introduce anything. The first sentence should introduce the integration.
Please revert.
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
Passing this one along to @c0ffeeca7, as she has been adjusting this documentation a lot lately with the goals of meeting requirements and matching copy created by our communications team. |
Proposed change
Attempt at improving Z-Wave integration introduction information + readability (and match the intro from the ZHA integration).
Type of change
current
branch).current
branch).next
branch).next
branch).Additional information
Checklist
current
branch.