Skip to content
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

feat(garageDoorOpener): initial attempt to support Novoferm ES423 with command ON/OFF instead of TOGGLE #146

Merged
merged 1 commit into from
Sep 15, 2024

Conversation

aure-olivier
Copy link
Contributor

@aure-olivier aure-olivier commented Feb 26, 2024

Salut @mgcrea,

Suite à cette issue #143, j'ai commencé à regarder un peu comment je pouvais modifier le GarageDoorOpener pour supporter mes portes de garage Novoferm ES423 (qui ne comprennent pas l'instruction TOGGLE, mais réagissent aux commandes ON/OFF).

J'ai fait un build custom et tester dans Homebridge, cela fonctionne plutôt pas mal.

Je n'ai pas trop d'idée sur comment intégrer ça autrement, garder le support TOGGLE pour ceux chez qui ça fonctionne et ajouter le ON/OFF pour les autres (comme moi).

PR créé pour ouvrir la discussion :)

Ce qui fonctionne chez moi:

  • Les portes sont reconnues dans Apple Home
  • Les portes peuvent s'ouvrir et se fermer depuis Apple Home
  • Si quelqu'un utilise la télécommande Novoferm (fournie avec les moteurs de porte) pour ouvrir/fermer une porte, l'état remonte bien dans Apple Home (si cette fermeture n'est pas faite pendant une action en cours côté Apple Home).
  • L'état ouvert/fermé est récupéré à l'initialisation de l'accessoire.

@mgcrea
Copy link
Owner

mgcrea commented Feb 27, 2024

Hello merci pour la PR! Je vais regarder pour faire fonctionner le code avec le setup actuel (TOGGLE) que j'ai à mon domicile, si pas possible de faire marcher les deux je rajouterai un setting pour sélectionner le bon mode ("on/off" / "toggle").

@aure-olivier
Copy link
Contributor Author

Salut @mgcrea ,
Tu as eu le temps de regarder le contenu et de voir ce qu'il est possible de faire ?
Ça serait vraiment cool d'avoir ce support 🙏

@aure-olivier
Copy link
Contributor Author

aure-olivier commented Aug 24, 2024

Salut tout le monde,

Du côté de Home Assistant, il y a la même problématique, et j'ai testé avec succès Tydom2mqtt avec la version temporaire fmartinou/tydom2mqtt:feature__183 (voir PR en cours: fmartinou/tydom2mqtt#199).

Pour ceux que ça intéresse 😉

@mgcrea mgcrea merged commit 7433f51 into mgcrea:master Sep 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants