SmartThingsCommunity / SmartThingsEdgeDrivers

Apache License 2.0
268 stars 453 forks source link

Support for compound devices and "grandchildren" #1356

Closed dljsjr closed 5 months ago

dljsjr commented 6 months ago

This refactor furthers the generalizations done previously for devices that provide multiple services to allow for not only mapping of services to capabilities or components, but also multiple device records.

This allows for us to support devices such as the Dymera up/down light.

Other changes:

github-actions[bot] commented 6 months ago

Duplicate profile check: Passed - no duplicate profiles detected.

github-actions[bot] commented 6 months ago

Test Results

   57 files  ±0    364 suites  ±0   0s :stopwatch: ±0s 1 757 tests ±0  1 757 :white_check_mark: ±0  0 :zzz: ±0  0 :x: ±0  3 051 runs  ±0  3 051 :white_check_mark: ±0  0 :zzz: ±0  0 :x: ±0 

Results for commit a3be7e3d. ± Comparison against base commit 2ace2f26.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 6 months ago

Minimum allowed coverage is 90%

Generated by :monkey: cobertura-action against 05ffd464bb59f8bdc44dfa4067076079de02b053

github-actions[bot] commented 6 months ago

Channel deleted.

dljsjr commented 6 months ago

There's a lot going on here.

Yeah. Adjusting the assumption that a Hue device service only exposes a single light service was a deep surgery 😢

github-actions[bot] commented 5 months ago

Invitation URL: https://bestow-regional.api.smartthings.com/invite/r3Myr5Nmwb2p