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.

Sign up for GitHub

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

Jump to bottom

Add nodeid to status update payload [feat] #492

Closed
chrish987 opened this issue Feb 7, 2021 · 2 comments · Fixed by #499
Closed

Add nodeid to status update payload [feat] #492

chrish987 opened this issue Feb 7, 2021 · 2 comments · Fixed by #499
Assignees
@robertsLando
Labels
enhancement New feature or request

Comments

@chrish987
Copy link

Status (awake/alseep) updates do not have the nodeid in the message payload.

You can determine this from the topic, but would be better if it was in the payload.

Topic example: zwavemqtt/SideDeck/MultiSensor6/status
Payload: {"time":1612658379924,"value":true,"status":"Awake"}
@chrish987 chrish987 added the enhancement New feature or request label Feb 7, 2021
@LordMike
Copy link
Contributor

LordMike commented Feb 8, 2021

Does this setting affect that?

image

@robertsLando
Copy link
Member

That's just for values

robertsLando added a commit that referenced this issue Feb 8, 2021
@robertsLando
fix: add node id to node status payload
aa259a4 
Fixes #492
@robertsLando robertsLando mentioned this issue Feb 8, 2021
Merged
robertsLando added a commit that referenced this issue Feb 8, 2021
@robertsLando
fix: add node id to node status payload (#499)
bf8a545 
* fix: add node id to node status payload

Fixes #492

* fix: lint issues
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@robertsLando robertsLando

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: add node id to node status payload zwave-js/zwave-js-ui
3 participants
@LordMike @robertsLando @chrish987