New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ajout ordre fil pilote conf -1 et conf -2 #42

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@bronco0
Copy link

bronco0 commented Dec 27, 2018

Bonjour,

J'ai ajouté les ordre Confort -1 et Confort -2 pour les fils pilotes.
Il faut peut être revoir la partie web pour l'arrangement des boutons des ordres.
J'ai utilisé l'icon confort, les -1 et -2 n'existe pas dans la font jeedom2.

3 Ticker permettent de générer le signal, un pour conf -1 pleine alternance (3s), un pour conf -2 pleine alternance (7s) et le dernier pour la base des 5 minutes.

J'ai du tricher pour ne pas modifier la variable etatFP pour qu'elle garde les valeur '1' ou '2'

@bronco0 bronco0 closed this Dec 27, 2018

@bronco0 bronco0 deleted the bronco0:fp_eco branch Dec 27, 2018

@bronco0 bronco0 restored the bronco0:fp_eco branch Dec 27, 2018

@Azema

This comment has been minimized.

Copy link
Collaborator

Azema commented Jan 7, 2019

Salut @bronco0

C'est un bon développement, même si je ne l'ai pas encore testé pour le moment.
Pourquoi as-tu fermé ta pull request ?

Manuel

@bronco0

This comment has been minimized.

Copy link

bronco0 commented Jan 7, 2019

Salut @Azema

C'est une erreur je vais refaire un pull requests.

Surtout j'ai corrigé quelques erreurs et testé le bon fonctionnement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment