Skip to content

Conversation

@tpmanley
Copy link
Contributor

Some hub connected DTHs were not setting the hubId when creating child
devices. This was resulting in the child devices not being shown in the
join screen in the app which had the side effect that they were not being
added to the room the user selected when enabling join.

https://smartthings.atlassian.net/browse/ICP-11101

Some hub connected DTHs were not setting the hubId when creating child
devices. This was resulting in the child devices not being shown in the
join screen in the app which had the side effect that they were not being
added to the room the user selected when enabling join.

https://smartthings.atlassian.net/browse/ICP-11101
@tpmanley tpmanley requested review from dkirker and greens September 26, 2019 22:09
isComponent: true, componentName: "ep$i", componentLabel: "Channel $i"
addChildDevice("Switch Child Device",
"${device.deviceNetworkId}-ep${i}",
device.hubId,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the main change. The rest is just formatting. Same in the other files

@greens
Copy link
Contributor

greens commented Sep 26, 2019

zwave-device-multichannel uses device.getHub().getId() and also has this bug so perhaps that's not correct and should also be changed to device.hubId?

@tpmanley
Copy link
Contributor Author

zwave-device-multichannel uses device.getHub().getId() and also has this bug so perhaps that's not correct and should also be changed to device.hubId?

Those two should be equivalent. It might be a different issue.

@tpmanley tpmanley merged commit f6f0b3f into SmartThingsCommunity:master Sep 27, 2019
@tpmanley tpmanley deleted the bugfix/hub-device-child branch September 27, 2019 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants