Race condition when two buttons reference the same light

[istathar]

There's a weird interaction between the knob being in toggle brightness/temperature mode and (now) having to add a button to turn the light on and off. In the photo below:

the light is off because I pressed big button, but dial still shows the light being on.

(An alternative design would be long-press on dial for toggling on/off and normal short-press being to toggle between brightness/temperature. This would eliminate the need to program two buttons to control one light which would be pretty cool)

[DavidVallaLope]

@cozybuild Seems that this would be tricky to fix. Right now each button stores one object of the action type that is referencing (and that class inherits from Core) but even if is the same device, the on_update only affects the object related to the key/dial used.

Do you know by any chance if StreamController allow us to manipulate directly objects? We might be able to get all the objects referencing to an specific IP to execute on_udpate() when a action that require an image update is triggered

As a side, we can implement a Long-Press on Dial to allow toggle or switch between settings as @istathar suggests but that would leave the race condition still there

[DavidVallaLope]

Silly me, I just observed we had Events already set up in the project. I will be implementing a fix using #6 PR since after supporting multiple devices this was still happening

[Cfomodz]

@DavidVallaLope Does this still need done or is it GTG in #6?