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

Asking for a "rebuild routes" on a particular node triggers error message #6357

Closed
3 tasks done
jfhautenauven opened this issue Sep 30, 2023 · 3 comments · Fixed by #6362
Closed
3 tasks done

Asking for a "rebuild routes" on a particular node triggers error message #6357

jfhautenauven opened this issue Sep 30, 2023 · 3 comments · Fixed by #6362
Labels
bug Something isn't working

Comments

@jfhautenauven
Copy link

Checklist

  • I am not using Home Assistant. Or: a developer has told me to come here.
  • I have checked the troubleshooting section and my problem is not described there.
  • I have read the changelog and my problem is not mentioned there.

Deploy method

Snap

Z-Wave JS UI version

9.0.3

ZwaveJS version

12.0.2

Describe the bug

When trying to ask for a rebuild route on a particular node (I mean by going via the node advanced options), there is an error stating something about SUC ... that ... literally SUCks :D

image

I guess it's something related to the recent revamp of the backend API.

To Reproduce

Go into the dashboard of devices
Select one victim amongst the available nodes
Expand the tab of that node
On the right upper part of that node tab, click on Advanced
Then Advanced Options
Then "Rebuild routes"

Expected behavior

Routes are getting rebuilt without error message.

Additional context

As a side node, rebuilding ALL the routes, so via the option on the right lower corner of screen (blue circle with lines) , so I mean rebuilding all the routes in the network seems to work.
BUT

The animation of the turning circle in the status column doesn't show as it used to.
No turning blue circle to indicate which nodes are still rebuilding their routes.

image

@jfhautenauven jfhautenauven added the bug Something isn't working label Sep 30, 2023
@robertsLando
Copy link
Member

@jfhautenauven By digging into this I think this is a driver error so something for @AlCalzone

ZWaveError: To assign a return route to the SUC (node ID 1), assignSUCReturnRoutes() must be used! (ZW0322)
    at ZWaveController.assignReturnRoutes (zwave-js/src/lib/controller/Controller.ts:4568:10)
    at ZWaveController.rebuildNodeRoutesInternal (zwave-js/src/lib/controller/Controller.ts:4233:18)
    at ZWaveController.rebuildNodeRoutes (zwave-js/src/lib/controller/Controller.ts:4106:11)
    at ZwaveClient.rebuildNodeRoutes (zwave-js-ui/lib/ZwaveClient.ts:2112:22)
    at ZwaveClient.callApi (zwave-js-ui/lib/ZwaveClient.ts:2693:20)
    at Socket.<anonymous> (zwave-js-ui/app.ts:503:26)

@robertsLando
Copy link
Member

robertsLando commented Oct 2, 2023

@jfhautenauven About the status not updating in UI.

There is a dedicated column that shows the rebuild routes status that I just noticed it's still named Heal. I think you edited the columns and that one is hidden because it should be on the right side of status.

I just pushed a change to name it Rebuild Routes instead but in order to have it back on your table simply click on reset table and/or click on columns and be sure the Heal column is checked

robertsLando referenced this issue in zwave-js/zwave-js-ui Oct 2, 2023
@robertsLando robertsLando transferred this issue from zwave-js/zwave-js-ui Oct 2, 2023
@jfhautenauven
Copy link
Author

@robertsLando : gonna check that column stuff, thanks for pointing it out :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants