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

[bug] Need a fullpage refresh to get association groups populated for new devices #1072

Closed
4 of 7 tasks
jmgiaever opened this issue Apr 17, 2021 · 2 comments · Fixed by #1076
Closed
4 of 7 tasks

[bug] Need a fullpage refresh to get association groups populated for new devices #1072

jmgiaever opened this issue Apr 17, 2021 · 2 comments · Fixed by #1076
Assignees
Labels
bug Something isn't working

Comments

@jmgiaever
Copy link
Contributor

jmgiaever commented Apr 17, 2021

Version

Checklist:

  • I am not using HomeAssistant. 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 problem was not mentioned there.

Build/Run method

  • Docker
  • PKG
  • Snap package
  • Manually built (git clone - npm install - npm run build )

zwavejs2mqtt: 3.3.0.b2f3402
zwave-js: 7.2.2

Describe the bug

When a new device is added the list of association groups is not populated.

Screenshot from 2021-04-17 08-58-34

You have to do a fullpage refresh to fill the list.

To Reproduce

Steps to reproduce the behavior:

  1. add new device
  2. wait for it to complete the inclusion
  3. go to groups and select the dropdown
  4. now do a refresh of the page and the list should be populated

Expected behavior

List to be populated with the groups

@jmgiaever jmgiaever added the bug Something isn't working label Apr 17, 2021
@robertsLando
Copy link
Member

Will look at this on Monday :)

@jmgiaever
Copy link
Contributor Author

PR i made for zwjs.

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