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

Add Z-Wave JS Heal Node wizard #9562

Merged
merged 5 commits into from
Jul 27, 2021
Merged

Conversation

cgarwood
Copy link
Member

Proposed change

Adds a dialog wizard to Z-Wave JS devices to heal the device.

Untitled.Project.mp4

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

this.entry_id!
);
if (network.controller.is_heal_network_active) {
this._status = "network-healing";
Copy link
Member

Choose a reason for hiding this comment

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

Since we are not waiting for the network fetch call, technically a user could now start a new heal when one is still active.

Copy link
Member Author

Choose a reason for hiding this comment

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

If you try to start a new heal while one is in progress the server will reject it, so our error handler will catch it.

@bramkragten bramkragten merged commit e12b194 into home-assistant:dev Jul 27, 2021
@cgarwood cgarwood deleted the zjs-heal-node branch July 27, 2021 22:18
@github-actions github-actions bot locked and limited conversation to collaborators Jul 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants