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

Add old_state to until parameter #6

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

XavierBerger
Copy link

@XavierBerger XavierBerger commented Aug 31, 2023

Bonjour Jean Loïc,

Voici une pull request pour notifier.py.

  • J'ai ajouté la possibilité de réagir sur le old_state dans la fonction until. Je m'en sert pour mon application de gestion de liste de courses multiples. Ca me permet très simplement de supprimer un notification qui aurai "popée" lorsque je passe devant un magasin en voiture sans que je n'ai l'intention de rester dans la zone pour faire mes courses. Dès que je sors de la zone, old_state contient le nom de la zone qui a "trigger" la notification et elle est supprimée.
  • J'ai aussi fait une petit optimisation sur le choix du type de notification à émettre.

Non proposé dans cette MR: j'ai mis en place un serie de tests automatiques pour toutes mes applications appdeamon (et donc y compris notifier.py) et c'est dispo ici. Si ça t'intéresse, je veux bien t'expliquer ce que j'ai mis en place et comment ça marche.

Enfin pour terminer, je suis viens de publier mon application de gestion de liste de courses multiples (qui utilise notifier.py) sur le forum HACF.fr dont les sources sont dispo sur mon github.

Merci pour tes partages

Xavier

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.

1 participant