Add Ramping time to dimmer channel for central command

fruggy83 / openocean

27 stars 11 forks source link

Add Ramping time to dimmer channel for central command #43

Closed dominikkv closed 5 years ago

dominikkv commented 5 years ago

Signed-off-by: Dominik Krickl-Vorreiter dominikkv@gmx.de closes #42

fruggy83 commented 5 years ago

Sieht gut aus, aber die EnOceanChannelDimmerConfig Klasse fehlt irgendwie 😉 Kannst du in der Klasse sicherstellen, dass sie kompatibel zu einer alten Konfig ist, also die ramping time per default auf 0 setzen. Du hast zwar schon eine Abfrage in der EEP Klasse, aber ein normaler int kann in Java doch nicht null sein, oder? Oder nutzt du einen Integer?

dominikkv commented 5 years ago

Hallo @fruggy83, danke, die fehlende Klasse habe ich hinzugefügt. Der PR war etwas chaotisch :-D Ich habe mir die Klasse bei den anderen Configs abgeschaut, da wird ein Integer verwendet. Ich hoffe das passt so.

fruggy83 commented 5 years ago

Thanks a lot 👍