Skip to content

Add documentation for Z-Wave credential management#45119

Draft
AlCalzone wants to merge 2 commits intohome-assistant:nextfrom
AlCalzone:zwave-credential-docs
Draft

Add documentation for Z-Wave credential management#45119
AlCalzone wants to merge 2 commits intohome-assistant:nextfrom
AlCalzone:zwave-credential-docs

Conversation

@AlCalzone
Copy link
Copy Markdown
Contributor

Proposed change

This PR adds documentation for new actions in the Z-Wave integration that serve as the foundation for the work-in-progress credential management feature.

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

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.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

Copilot AI review requested due to automatic review settings April 30, 2026 18:34
@AlCalzone AlCalzone requested a review from a team as a code owner April 30, 2026 18:34
@home-assistant home-assistant Bot added has-parent This PR has a parent PR in another repo next This PR goes into the next branch labels Apr 30, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 30, 2026

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit ccadd76
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/69f863490ebd36000850d17b
😎 Deploy Preview https://deploy-preview-45119--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds documentation to the Z-Wave JS integration page for the new user/credential management actions that underpin the in-progress credential management feature (core PR home-assistant/core#168360).

Changes:

  • Documented new “user and credential management” actions (set/delete users, query capabilities/users, set/delete credentials).
  • Added capability/user type/credential type reference tables and YAML usage examples.
  • Positioned the new docs alongside the existing legacy lock usercode actions.

Comment thread source/_integrations/zwave_js.markdown
Comment thread source/_integrations/zwave_js.markdown Outdated
Comment on lines +786 to +794
The `zwave_js.set_user` action creates or updates a user on the lock. If you omit `user_id`, the integration assigns the first available slot. The action returns the assigned `user_id`.

| Data attribute | Required | Description |
| ----------------- | -------- | ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
| `user_id` | no | User slot index (1-based). Defaults to the first available slot. |
| `user_name` | no | Display name for the user. Maximum length is reported by `get_credential_capabilities`. When omitted, the existing name is preserved on update or left empty on create. |
| `user_type` | no | Type of user to create. See [user types](#user-types) below. Defaults to the existing value on update, or `general` on create. |
| `credential_rule` | no | How many credentials must be presented to unlock. One of `single`, `dual`, or `triple`. Defaults to the existing value on update, or the lock's default (typically `single`) on create. |
| `active` | no | Whether the user is active. Inactive users exist on the lock but cannot unlock with their credentials until reactivated. Defaults to `true`. |
Copy link

Copilot AI Apr 30, 2026

Choose a reason for hiding this comment

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

The data-attribute tables for these new actions don’t document how to target the lock (for example, entity_id), even though every other action on this page includes target selection in the table and your YAML examples rely on target.entity_id. To avoid confusion, document the targeting fields (or add a short sentence stating these actions must be called with a lock target, consistent with other action docs here).

Copilot uses AI. Check for mistakes.
Comment thread source/_integrations/zwave_js.markdown Outdated
Comment thread source/_integrations/zwave_js.markdown Outdated
Copy link
Copy Markdown
Contributor

@c0ffeeca7 c0ffeeca7 left a comment

Choose a reason for hiding this comment

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

For set_user, set_credential, get_credential_capabilities, would it be worth adding a response example, like https://github.com/home-assistant/home-assistant.io/pull/45119/changes#diff-834a19fc622b88161ff1f89bf5abed6f1000f0f064170c77ecebda5e688fb042L759.

@home-assistant home-assistant Bot marked this pull request as draft May 4, 2026 07:32
@home-assistant
Copy link
Copy Markdown

home-assistant Bot commented May 4, 2026

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-parent This PR has a parent PR in another repo next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants