Skip to content

Conversation

Hedda
Copy link
Contributor

@Hedda Hedda commented Aug 22, 2025

Proposed change

Attempt at improving Z-Wave integration introduction information + readability (and match the intro from the ZHA integration).

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

  • Link to parent pull request in the codebase:
  • Link to parent pull request in the Brands repository:
  • This PR fixes or closes issue: fixes #

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
  • The documentation follows the Home Assistant documentation standards.

Update Z-Wave integration introduction to match the into from the ZHA integration.
@Hedda Hedda requested a review from a team as a code owner August 22, 2025 09:28
@home-assistant home-assistant bot added the current This PR goes into the current branch label Aug 22, 2025
Copy link

netlify bot commented Aug 22, 2025

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 8954fdf
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/68a83a439f6cfb00086d9e0c
😎 Deploy Preview https://deploy-preview-40562--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

misspelling, changed refered to referred
Comment on lines +60 to +75
- [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/)
Copy link
Member

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).
Copy link
Member

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.

@home-assistant home-assistant bot marked this pull request as draft August 27, 2025 08:33
@home-assistant
Copy link

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@frenck frenck requested a review from c0ffeeca7 August 27, 2025 08:34
@frenck
Copy link
Member

frenck commented Aug 27, 2025

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
current This PR goes into the current branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants