search

francispoisson / hilo

Hilo integration for Home Assistant
The Unlicense
26 stars 8 forks source link

Refactor the hiloapi #31

Closed valleedelisle closed 2 years ago

valleedelisle commented 2 years ago

We should avoid exec at all cost as it's not safe. setattr will do a fine job in this specific use case.

Wrapping requests under the same method will make it more reliable.

Unless we have a specific usecase, we shouldn't have a class in a class.

Removing repetitive code.

Also, we need to turn the lights on before setting intensity otherwise the intensity call is not taken into account.

fixes #25

valleedelisle commented 2 years ago

Idéalement on utiliserait async mais pour le moment j'ai pas l'temps de me casser la tête avec ça. En attendant, ça semble résourdre tous les bugs que j'ai vécu à date.

francispoisson commented 2 years ago

Super, merci! Ça prenait un professionnel pour réviser ça ! 👍

valleedelisle commented 2 years ago

@francispoisson Je suis vraiment impressionné par ton travail, t'as fait preuve d'une très grande débrouillardise, surtout que ça semble être un de tes premiers projet python. Lâche pas et n'hésite pas à me pinger si j'peux aider.