diyhue / diyHue

Main diyHue software repo
https://diyhue.org/
Other
1.52k stars 274 forks source link

Always include speed in scenes for api V2 #960

Closed jamesremuscat closed 7 months ago

jamesremuscat commented 7 months ago

The Hue v2 API mandates speed in scene objects (even though the palette entry, to which the speed supposedly relates, is optional).

aiohue at the very least has trouble parsing scenes where speed is not specified.

This PR modifies the Scene object to always include speed in its V2 API representation, and pulls out the default value into a constant to use across both constructor and save methods.

mariusmotea commented 7 months ago

Thanks for the fix.