En effet je ne songeais pas à ces éventuel retours… un nid à problème cette fonction 
Moi je peux faire des tests en installant un Gladys sur mon NAS, après je n’ai pas de tablette à disposition pour ça. En gros faudrait me dire quoi tester ^^
Si tu génères une image, je peux tester sur un des RPis annexe. Par contre, n’étant pas chez moi, je ne saurai pas tester le verrouillage tablette en direct…
Ça pose un problème si c’est installé sur la dernière version de RPi OS Bookworm ?
@Lokkye Merci pour la review, j’ai tout fixé 
Preneur d’autres reviews !
@GBoulvin non tu peux tester sur ce que tu veux, normalement pas de soucis. Je te tiens au courant quand une image est prête !
1 « J'aime »
Je me rend compte que j’avais complètement oublié l’action de scènes pour passer l’alarme en mode « armée/désarmée/etc… »
Pratique pour faire une scène « quand je clique sur un bouton » => « armer l’alarme »
Je bosse dessus!
1 « J'aime »
Bon je vous ai fais une petite vidéo pour vous faire une démonstration du mode alarme de bout en bout, comme ça ça permet à ceux qui ne peuvent pas tester de faire quand même des retours
(Petit bug à 3:11 à ne pas prendre en compte)
(La vidéo est non-repertoriée, à ne pas partager, je la supprimerais une fois qu’on a fini la review)
Pour ceux qui veulent tester
Une image Docker est disponible !
Attention, ne pas lancer sur votre production, cette fonctionnalité modifie la base de donnée
gladysassistant/gladys:alarm-mode
4 « J'aime »
Ca m’a l’air pas mal. 
Peut-être que tu l’as prévu mais faudrait peut-être empêcher le navigateur de sauvegarder le code d’accès.
Si j’ai le temps j’essaierai ton image.
J’ai ptet loupé, mais est-ce quet u as aussi fait en sorte d’avoir un retour quand on se trompe plusieurs fois sur le code ?
Quelqu’un qui essaye de désactiver plusieurs fois et qui se trompe X fois, peu-être mettre un timer pour retenter de taper le code et avec une scène qui permet ensuite d’envoyer un message ou je ne sais quoi ?
1 « J'aime »
Je ne sais pas si ça a déja été évoqué mais il pourrait aussi y avoir la notion de code factice ? C’est un code qui déclenche une action particulière (généralement appeler les secours). Le scénario -assez pessimiste- c’est que quelqu’un t’escorte méchament chez toi en te disant de désarmer l’alarme ou de lui donner le code. A ce moment là tu donnes / tapes le mauvais code, l’alarme se désarme mais surtout une action spécifique est engagée.
Je viens de tester l’image docker, je trouve le fonctionnement très adapté à ce que l’on veut. Plusieurs remarques :
-
le placeholder du code de l’alarme indique qu’il faut 6 chiffres > à modifier si on peut mettre autant de caractères que l’on veut.
-
il n’y a pas de retour visuel lorsqu’on sauvegarde la configuration de la maison. C’est peut être le cas dans d’autres onglets de paramètres, mais je trouve que ça manque un peu. Genre un petit toast / alert pour confirmer la sauvegarde.
-
fenêtre du code en mode tablette : je trouve le clavier un peu petit, et je l’aurais bien vu avec des touches plus carrées. Manière de faire le code viteuf avec mes gros doigts ^^
-
je ne suis pas fan du design du widget alarme, en tout cas des interactions que l’on peut avoir avec.
Le fait que le bouton ARMER soit bleu alors que les autres sontoutline, ça prête un peu à confusion je trouve.
De même, le bouton Armement partiel et Panique une fois cliqués passent en styledisabled, mais ce n’est pas hyper flagrant. J’aurais volontiers vu le comportement suivant : -
tous les boutons sont en style outline
-
lors d’un appui sur Armer / Panique / Partiel, alors le bouton passe en style active, tandis que le Désarmer reste en outline.
-
le clic sur le bouton Désarmer fait repasser les boutons en outline.
Voila pour mon retour
J’ai voulu tester mais l’installation de Gladys a fait planter mon pi qui tournait sous Bookworm.
Pas sûr que ça aie un lien mais l’installation sous bullseye, elle, s’est bien passée…
Tu parles bien du docker ? Ou l’installation manuelle ?
Super, c’est simple et ça couvre en même temps beaucoup de cas d’usages
Dans l’action Passer l’alarme en mode => Armé, comment gérer le délai ?
Est-ce qu’il faut ajouter avant un temps d’attente dans la scène ou est-ce qu’on pourrait Passer l’alarme en mode => Armement (qui prendrait en compte le délai configuré pour la maison) ?
Installation de Docker ok
Mais une fois le Docker pull terminé, le Pi s’est planté au démarrage du container… Plus d’accès en SSH. N’étant pas chez moi, le redémarrage manuel attendra 
Du coup c’est étrange, docker que ce soit bullseye / bookworm ne devrait pas te faire planter ton pi 
D’ou ma question, etrange
Oui bonne idée ! Je corrige demain.
Bien vu, je vais ajouter un rate limit et un message associé au cas ou il y a trop de tentatives !
ça me parait un peu beaucoup pour cette première release Restons dans le cas d’usage basique pour l’instant !
Bien vu ! Je vais corriger.
Ok bien vu ! Je vais corriger
Ok, par curiosité tu es sur quelle taille d’écran ? (résolution/et taille?)
Que je fasse du testing en réel
Ok intéressant, j’étais parti sur le point de vue inverse, pour moi l’idée c’est que le bouton soit actif quand clickable, ou sinon pas actif. Mais tu as peut-être raison, si ça t’as paru bizarre…
Fais peut-être un autre sujet ? J’avoue que je n’ai jamais testé Gladys sous bookworm, je pense pas que ce soit lié à la fonctionnalité alarme 
Il n’y a pas de délai dans l’action de scène
PS: Si possible, j’aimerais bien faire la release de cette fonctionnalité lundi (sauf si un gros bug est découvert), donc le plus vite j’ai des retours le mieux c’est 
Merci à ceux qui testent !
4 « J'aime »
Et possibilite davoir un declancheur sur ça dans les scenes ?
Genre lutilisateur se trompe, ou un mec tente x fois cela permet de faire des chose (envoie une image de la camera par exemple + telegram)
1 « J'aime »
Merci @pierre-gilles pour tes remarques
Alors je n’ai pas testé sur une tablette mais depuis mon ordi en changeant la taille de la fenêtre pour simuler différentes tailles d’écran. C’est peut être pour ça que j’ai trouvé le clavier à code un peu petit ?
Pour le truc du code factice, je l’ai surtout écrit pour soumettre l’idée quelque part, mais ce n’est en effet pas super utile sur une première release.
Je laisse chacun donner son avis sur le style des boutons cliquables, manière qu’on arrive à un consensus à ce niveau
Ayant une de nos connaissances qui a été victime de home jacking et qui a été obligée de désactiver son alarme, c’est carrément une super idée 
…même pas implémentée dans mon alarme myfox (somfy je sais pas) ! 
Dans la prochaine release effectivement ce serait top d’avoir 2 code, un normal, un home-jacking !
2 « J'aime »
Pourrais ton avoir un visuel des deux styles ? On pourrais donner plus facilement notre avis pour les no css loveur
En revanche, je trouve l’idée vraiment cool !
1 « J'aime »
Voilà (à l’arrache) ce que je voulais dire :
Par défaut, tout est comme ça :
Une fois que j’ai cliqué sur ARMER et que l’alarme l’est, le bouton apparait comme ça :
Pareil pour les autres boutons :
Un appui sur DERSARMER fait revenir l’état comme au départ.
1 « J'aime »
Je pense que par défaut le background de désarmée doit etre set une alarme a forcement un etat
Mais sinon oui cest plus visuel de quoi est actif ou non ^^.