# Review des PR - J'ai besoin de vous! **URL:** \ **Category:** Développement\ **Created:** [Janvier 18, 2021, 4:41 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030 "2021-01-18T16:41:56Z")\ **Posts on this page:** 20\ **Page:** 2
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Mars 12, 2021, 9:46 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/21 "2021-03-12T09:46:03Z")
Pour info, j’ai lancé un build dev sur master avec les changements sur le Philips Hue + la propagation des messages de l’utilisateur dans d’autres onglets 🙂 ---
**Author:** ![AlexTrovato](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/alextrovato/32/10705_2.png) [@AlexTrovato](https://community.gladysassistant.com/u/AlexTrovato)\ **Post date:** [Mars 14, 2021, 9:24 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/22 "2021-03-14T09:24:25Z")
J’ai vu passé ça ce matin [https://github.com/GladysAssistant/Gladys/pull/1098](https://github.com/GladysAssistant/Gladys/pull/1098) Ok, 165 fichiers, mais il y en a au moins 70 pour lister les devices compatibles. Donc courage !!! ---
**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\ **Post date:** [Mars 21, 2021, 7:45 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/23 "2021-03-21T07:45:16Z")
Les triggers sunset sunrise ne fonctionne pas chez moi, quelqu’un a testé en réel ? ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Mars 21, 2021, 11:04 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/24 "2021-03-21T11:04:37Z")
Normalement @AlexTrovato avait testé je crois ! Ca ne marche pas du tout ? ---
**Author:** ![AlexTrovato](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/alextrovato/32/10705_2.png) [@AlexTrovato](https://community.gladysassistant.com/u/AlexTrovato)\ **Post date:** [Mars 21, 2021, 11:06 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/25 "2021-03-21T11:06:08Z")
lol j’avais éteint ma machine, l’aube est arrivé, mais Gladys ne tournait pas 😕 ---
**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\ **Post date:** [Mars 21, 2021, 11:07 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/26 "2021-03-21T11:07:52Z")
J’ai créér une scene toute simple hier soir: Envoi d’un message au lever du soleil. Rien dans le chat ce matin ( la localisation est bonne ) ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Mars 21, 2021, 11:08 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/27 "2021-03-21T11:08:36Z")
Ah merde… my bad c’est l’auteur de la PR qui avait testé en réel: ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/3/38a7a0271fb798c0b03cbd74132ac8a54f338047.jpeg) ---
**Author:** ![AlexTrovato](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/alextrovato/32/10705_2.png) [@AlexTrovato](https://community.gladysassistant.com/u/AlexTrovato)\ **Post date:** [Mars 21, 2021, 11:09 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/28 "2021-03-21T11:09:19Z")
Je tente de préparer un test pour ce soir, en esperant voir des logs. ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Mars 21, 2021, 11:09 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/29 "2021-03-21T11:09:39Z")
Je me fais un test aussi ! ---
**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\ **Post date:** [Mars 21, 2021, 11:10 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/30 "2021-03-21T11:10:28Z")
Aucune trace pour moi, le truc bizarre c’est qui si je lance la scene en manuel, le message est bien reçu. Au pire pour tester faut changer la localisation de la maison ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Mars 21, 2021, 12:16 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/31 "2021-03-21T12:16:52Z")
J’ai mis une scène pour demain matin 🙂 Réponse à 6:20 ! ---
**Author:** ![lmilcent](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/lmilcent/32/7023_2.png) [@lmilcent](https://community.gladysassistant.com/u/lmilcent)\ **Post date:** [Mars 21, 2021, 12:49 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/32 "2021-03-21T12:49:39Z")
La fonctionnalité la plus longue à debugger 😂 Faut être patient ! ---
**Author:** ![AlexTrovato](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/alextrovato/32/10705_2.png) [@AlexTrovato](https://community.gladysassistant.com/u/AlexTrovato)\ **Post date:** [Mars 21, 2021, 2:04 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/33 "2021-03-21T14:04:24Z")
lol 1ere réponse ce soir, ce sera la surprise à la maison (avec un prime un “t’as fait quoi encore ?” de Madame !!) ---
**Author:** ![AlexTrovato](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/alextrovato/32/10705_2.png) [@AlexTrovato](https://community.gladysassistant.com/u/AlexTrovato)\ **Post date:** [Mars 21, 2021, 6:13 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/34 "2021-03-21T18:13:15Z")
J’ai eu une scene qui s’est lancée à 18h45, heure du couché. ---
**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\ **Post date:** [Mars 21, 2021, 6:18 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/35 "2021-03-21T18:18:23Z")
Je confirme sunset c’est ok, je remet le sunrise pour demain ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Mars 22, 2021, 1:30 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/36 "2021-03-22T01:30:23Z")
Je confirme, sunrise non fonctionnel, ça n’a pas marché ce matin! @Lokkye une idée? 🙂 Edit: j’investigue, et déjà il y a un problème de timezone. Chez moi (en GMT+8), l’heure utilisée est l’heure de Londres, et du coup à minuit (dans ma timezone), Gladys calcule le sunrise/sunset à l’heure de Londres, donc la veille. Cf: > ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Mars 22, 2021, 5:28 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/37 "2021-03-22T05:28:18Z")
Ok, ça m’a pris du temps à comprendre, mais je pense être sur la bonne voie: [https://github.com/GladysAssistant/Gladys/pull/1103](https://github.com/GladysAssistant/Gladys/pull/1103) J’avance mais c’est pas encore exactement ça ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Mars 22, 2021, 7:47 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/38 "2021-03-22T07:47:39Z")
Pour info, j’ai review toutes les PR qu’il y avait en attente (selon ce que je vois côté Github). C’était éprouvant 😅 Si je n’ai pas review une PR, c’est que je ne la vois pas, donc merci de mettre un message sur le forum pour me prévenir 🙂 ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Mars 22, 2021, 7:57 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/39 "2021-03-22T07:57:27Z")
Pour l’histoire de la timezone, c’est normalement fixé et merge sur master et buildé sur docker sur le tag “dev”. J’ai le sunset qui va bientôt tourner chez moi. Demain matin je testerais le sunrise. En tout cas les logs ont l’air de confirmer que c’est corrigé (j’ai rajouté des logs plus verbose): ![Screenshot 2021-03-22 at 15.56.40](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/d/dfebf8593c4049c0c153fc1c806284a4faeb2da1.png) ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Mars 22, 2021, 10:31 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/40 "2021-03-22T10:31:16Z")
Sunset ok! ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/f/f75b50960b169e79e004ea937289799eb432cda8.jpeg) À demain pour le sunrise 🙂 [Page précédente](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030.md?page=1) [Page suivante](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030.md?page=3)