Feature Thermostat complète

Non ca n’existe pas. Tu pourrais faire une demande de fonctionnalité poir cela.

Moi je ferais un thermomètre virtuel moyenne et dans une scène tu récupère les valeurs de chaque thermomètre ensuite tu applique un calcul pour obtenir la moyenne

Voilà, testé.

En résumé, c’est vraiment, vraiment, vraiment top !

Deux choses qui m’ont posé problème. Testé sur mobile Android uniquement :

  1. Dans le mode planning, si je clique sur l’heure, le clavier n’apparaît pas, ni le choix de l’heure (et l’icône de l’horloge est trop peu visible en mode sombre)

  1. Si je mets en ‹ Arret ›, j’ai beau ‹ revenir au planning ›, le bouton ne se met pas à jour (il devrait revenir en mode éco ou nuit ou… (En fait, à cette heure ci, je n’avais rien défini donc je me demande en écrivant ces lignes ce qui est défini par défaut. Eco ? Ah, non. En allant revoir, c’est ‹ arrêt › par défaut donc je suppose que tout est normal…)

Je suis tout enthousiaste et impatient de voir ce module en production, là ! Bravo, c’est vraiment top (je l’ai déjà dit ?)

Merci beaucoup pour ton retour précieux…Claude a transpiré quelquefois…

Ok pour le point 1 je ferais (Claude) une correction.

Pour le point 2, oui si tu n’as rien défini dans ton planning c’est normal. Après du moment que tu as appuyé sur le bouton comme le message reste cela t’a peut etre perturbé. Je réfléchi a ce que je pourrais faire.

Merci encore pour ton retour

Tu as raison, c’est bien ça le problèmes. Le message devrait repasser à ‹ arret ›, comme pour les autres modes :wink:

@GBoulvin, c’est quoi ton navigateur sur ton mobile Android ?
Claude me dit que cela provient sûrement du navigateur.

Est-ce que si je mets le sélecteur comme dans les scènes, cela conviendrait ?

C’est Chrome.

Le sélecteur actuel est très bien, j’aurais juste aimé pouvoir écrire l’heure manuellement, au clavier, ou peut-être simplement que le sélecteur s’ouvre au clic sur l’heure et pas uniquement sur le logo.

Je peux retester en fin de journée avec un autre navigateur si tu veux…

Ah ok j’avais pas du tout compris cela.

Je veux bien que tu reteste avec un autre navigateur si c’est possible pour toi.
Merci

@Will_71 Tiens-moi au courant dès que tu penses que c’est prêt à partir ! :grinning_face_with_smiling_eyes: L’idée serait de viser une release la semaine prochaine au plus tard, pour que je sois encore là pendant les deux premières semaines de vie de la fonctionnalité. Pour info, il me reste entre 9 et 19 jours de présence sur Gladys, selon la date de l’accouchement !

Si ça te va, je propose de merge aujourd’hui sur master, et de faire un build :dev. Si il y a des retours pendant le week-end, je mergerai les retours lundi, sinon ça part lundi en production :slight_smile:

Je suis un peu sous le feu avec mon taf aujourd’hui mais je peux avancer ce week-end dessus. Il reste les 2 points que @GBoulvin a remonté.

@Will_71 J’ai lancé mon Claude pour faire une analyse des retours de @GBoulvin et ses retours sont légitimes. Le diagnostic complet est dispo ici : feat(thermostat): add a thermostat integration with dashboard widget by William-De71 · Pull Request #2988 · GladysAssistant/Gladys · GitHub

Si tu n’es pas dispo aujourd’hui, je te propose de faire moi-même les correctifs, puis de merger sur master, et ensuite si tu as d’autres correctifs à faire ce week-end, tu peux refaire d’autres PRs et ça partira lundi, tu en penses quoi ? :slight_smile:

Oui vas y, pour aujourd’hui ca permettra d’avancer car je ne pourrais pas m’en occuper.
Je pourrais regarder plus en détails ce week-end mais pour moi il restait que ces 2 points.

Une fois ton merge sur master je pourrais préparer la suite aussi avec.la gestion des fils pilotes…PRs prêt de mon côté a être rebase et ensuite a testé en réel.

Salut tout le monde !

Ce sujet est désormais en cours de développement.

Une PR a été ouverte pour corriger le bouton « Revenir au planning » sur un créneau Arrêt, et le sélecteur d’heure du planning sur Android :

N’hésitez pas à suivre la PR, à tester (optionnel, surtout pour les petites demandes) et à faire vos retours ici si besoin.

J’en suis au moins à ma 3ème passe :joy:

J’ai fait faire une passe UX à Claude en réel + plusieurs code review par Fable 5.1.

Ça a mouliné pendant 45 minutes et fait plein de screenshots, et il a trouvé quelques bugs.

Il corrige !

Casse pas tout mon travail :sob:.

Je rigole, si on peut corriger d’autres bugs qu’on a pas vu et que le thermostat soit parfait​:+1:

J’espère que j’ai rien cassé :joy:

Tiens moi au courant quand tu as avancé/terminé que je puisse testé de mon côte

Ce que je vais faire, c’est que de toute façon je vais merger sur master dès que je suis satisfait aujourd’hui, comme ça je peux faire un build qui regroupe tous les gros changements de cette semaine, et ensuite si tu as des retours, tu pourras me faire des petites PRs correctives ce week-end que j’intégrerai lundi :slight_smile:

C’est mergé sur master !