Skip to content
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

Notify user about factory reset of end devices #6

Closed
marcelveldt opened this issue Jul 13, 2023 · 1 comment · Fixed by home-assistant/core#101362 or home-assistant/core#104291
Labels
new feature New feature or request

Comments

@marcelveldt
Copy link
Collaborator

marcelveldt commented Jul 13, 2023

The specifications mandate this:

A controlling node receiving the Device Reset Locally Notification Command MUST indicate to the
end user that the node has been reset and left the Z-Wave network.

To accomodate, v11 of node-zwave-js has added a parameter that indicates why a node was removed:
https://zwave-js.github.io/node-zwave-js/#/api/controller?id=quotnode-removedquot

When this reason is RemoveNodeReason.Reset, HA must notify the user. Other reasons, like ProxyExcluded or ProxyReplaced may be interesting too, but not mandatory.

@marcelveldt marcelveldt changed the title Z-Wave JS: When a node was factory reset, the user must be notified of this When a node was factory reset, the user must be notified of this Jul 13, 2023
@marcelveldt marcelveldt added new feature New feature or request and removed integration: zwave_js labels Jul 13, 2023
@marcelveldt marcelveldt changed the title When a node was factory reset, the user must be notified of this Notify user about factory reset of end devices Jul 13, 2023
@raman325 raman325 moved this from Later to Done in Z-Wave JS: Road to Certification Oct 5, 2023
@raman325 raman325 closed this as completed Oct 5, 2023
@AlCalzone
Copy link
Member

One remaining issue: The integration must be reloaded before the device gets removed from HA. This fails this requirement:

Has the DUT removed this node (ID = 3) from its Z-Wave nodes list? (see Requirement Number CL:005A.01.52.01.1)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature New feature or request
Projects
3 participants