# Philips Hue améliorations du scan **URL:** \ **Category:** Développement\ **Tags:** philips-hue\ **Created:** [Octobre 28, 2023, 9:49 UTC](https://community.gladysassistant.com/t/philips-hue-ameliorations-du-scan/8427 "2023-10-28T21:49:09Z")\ **Posts on this page:** 12\ **Page:** 1
**Author:** ![cicoub13](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/cicoub13/32/4482_2.png) [@cicoub13](https://community.gladysassistant.com/u/cicoub13)\ **Post date:** [Octobre 28, 2023, 9:49 UTC](https://community.gladysassistant.com/t/philips-hue-ameliorations-du-scan/8427/1 "2023-10-28T21:49:09Z")
Bonjour, J’ai commencé à travailler sur l’amélioration du scan Philips Hue en : - ajoutant le scan N-UPnP (mais cela nécessite une requête supplémentaire pour récupérer le serial) - ajoutant la possibilité de rentrer une adresse IP manuellement depuis le Front J’aurai besoin d’aide sur 3 points : - le design du form front. J’ai essayé dans une card, dans un form, c’est toujours moche 🤢 - je n’arrive pas à setter une valeur `bridge.ipaddress` depuis un input et l’utiliser plus tard. Ça paraît un basique mais je bloque (dans `SetupTab.jsx`) - je ne peux pas tester en vrai car plus de pont Philips Hue 😐 La branche est là : [feat(philips-hue): N-UPnP Search + manual configuration from IP · cicoub13/Gladys@df3fe52 · GitHub](https://github.com/cicoub13/Gladys/commit/df3fe52d8e28d14e7b2e9713b914e149d7f0cc24) ![CleanShot 2023-10-28 at 23.50.30@2x](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/7/76d635bf87ff8bc46aa5ebecd63f937fd6c6ef10.png) ---
**Author:** ![cicoub13](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/cicoub13/32/4482_2.png) [@cicoub13](https://community.gladysassistant.com/u/cicoub13)\ **Post date:** [Octobre 28, 2023, 10:12 UTC](https://community.gladysassistant.com/t/philips-hue-ameliorations-du-scan/8427/2 "2023-10-28T22:12:19Z")
Ah oui, il est noté dans la documentation que le scan N-UPnP fonctionne uniquement si le pont à été enregistré dans l’application officielle. Il faudra le rajouter dans notre documentation comme pré-requis ---
**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:** [Octobre 30, 2023, 8:29 UTC](https://community.gladysassistant.com/t/philips-hue-ameliorations-du-scan/8427/3 "2023-10-30T08:29:48Z")
Salut @cicoub13 🙂 Trop cool, merci pour la PR ! > [@cicoub13](#): > > le design du form front. J’ai essayé dans une card, dans un form, c’est toujours moche 🤢 Tu pourrais peut-être reprendre le format qu’on a sur les autres intégrations ? Un bouton « + » à côté de « Recherche sur le réseau », qui ajouterait une card (inspire toi tu design des autres cards d’appareils) > [@cicoub13](#): > > je n’arrive pas à setter une valeur `bridge.ipaddress` depuis un input et l’utiliser plus tard. Ça paraît un basique mais je bloque (dans `SetupTab.jsx`) Je te conseille de lire du code d’autres endroits dans Gladys pour comprendre 🙂 Exemple pour la page dashboard, on créé une fonction: [Gladys/front/src/routes/dashboard/index.js at master · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/blob/master/front/src/routes/dashboard/index.js#L11) Qui est ensuite passé en props au component : [Gladys/front/src/routes/dashboard/index.js at master · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/blob/master/front/src/routes/dashboard/index.js#L222) Si tu es débutant avec Preact/React, peut-être que faire un tutoriel pourrait t’aider ? La doc de React est super bien faite et il y a un tutoriel interactif : [Quick Start – React](https://react.dev/learn) > [@cicoub13](#): > > je ne peux pas tester en vrai car plus de pont Philips Hue 😐 Pour ça on trouvera quelqu’un qui en a un dès que la PR est prête ! ---
**Author:** ![Tlse-vins](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/tlse-vins/32/6054_2.png) [@Tlse-vins](https://community.gladysassistant.com/u/Tlse-vins)\ **Post date:** [Octobre 31, 2023, 9:22 UTC](https://community.gladysassistant.com/t/philips-hue-ameliorations-du-scan/8427/4 "2023-10-31T09:22:48Z")
Bonne nouvelle, J’avais fait remonter le problème de détection il y a longtemps. Chez certain cela fonctionnais bien et chez d’autre pas du tout. ---
**Author:** ![cicoub13](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/cicoub13/32/4482_2.png) [@cicoub13](https://community.gladysassistant.com/u/cicoub13)\ **Post date:** [Octobre 31, 2023, 10:42 UTC](https://community.gladysassistant.com/t/philips-hue-ameliorations-du-scan/8427/5 "2023-10-31T10:42:25Z")
Le front et le back fonctionnent 😅 Une image est disponible pour tester : - scan et ajout fonctionne toujours - scan N-UPnP fonctionne pour les gens qui avaient des problèmes précédemment - ajout via IP manuel fonctionne - pilotage de devices (lampes) fonctionne toujours (car j’ai touché un peu le stockage des configurations des bridges) `docker pull cicoub13/gladys:philips-hue-improve-scan` ---
**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:** [Novembre 1, 2023, 8:42 UTC](https://community.gladysassistant.com/t/philips-hue-ameliorations-du-scan/8427/6 "2023-11-01T08:42:50Z")
@cicoub13 Super cool, content que tu ai pu trouver ce qui ne marchait pas niveau front 🙂 Limite met un message sur le forum EN peut-être que l’utilisateur qui avait le souci pourra faire du test aussi ! ---
**Author:** ![cicoub13](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/cicoub13/32/4482_2.png) [@cicoub13](https://community.gladysassistant.com/u/cicoub13)\ **Post date:** [Novembre 1, 2023, 12:28 UTC](https://community.gladysassistant.com/t/philips-hue-ameliorations-du-scan/8427/7 "2023-11-01T12:28:41Z")
Fait : [Phillips Hue Not findable - #9 par cicoub13 - Configuration - Gladys Community (EN)](https://en-community.gladysassistant.com/t/phillips-hue-not-findable/137/9?u=cicoub13) ---
**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:** [Novembre 1, 2023, 12:29 UTC](https://community.gladysassistant.com/t/philips-hue-ameliorations-du-scan/8427/8 "2023-11-01T12:29:39Z")
Bien vu pour le tutoriel pour lancer une image Docker ! Merci d’avoir pris le temps 🙏 ---
**Author:** ![cicoub13](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/cicoub13/32/4482_2.png) [@cicoub13](https://community.gladysassistant.com/u/cicoub13)\ **Post date:** [Novembre 9, 2023, 5:22 UTC](https://community.gladysassistant.com/t/philips-hue-ameliorations-du-scan/8427/9 "2023-11-09T17:22:21Z")
PR prête (testée par un utilisateur anglais) > > > \### Pull Request check-list > > To ensure your Pull Request can be accepted as fa…st as possible, make sure to review and check all of these items: > > \- \[x\] If your changes affects code, did your write the tests? > \- \[x\] Are tests passing? (\`npm test\` on both front/server) > \- \[x\] Is the linter passing? (\`npm run eslint\` on both front/server) > \- \[x\] Did you run prettier? (\`npm run prettier\` on both front/server) > \- \[x\] If you are adding a new features/services, did you run integration comparator? (\`npm run compare-translations\` on front) > \- \[x\] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community (\[french forum\](https://community.gladysassistant.com/)/\[english forum\](https://en-community.gladysassistant.com/)) for testing before merging. > \- \[x\] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code) > \- \[\] If you are adding a new features/services which needs explanation, did you modify the user documentation? See \[the GitHub repo\](https://github.com/GladysAssistant/v4-website) and the \[website\](https://gladysassistant.com). > \- \[\] Did you add fake requests data for the demo mode (\`front/src/config/demo.js\`) so that the demo website is working without a backend? (if needed) See \[https://demo.gladysassistant.com\](https://demo.gladysassistant.com). > > NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open. > > \### Description of change > > Following multiple issues about Philips Hue discovery, this PR allow to: > \- Enable both N-UPnP and UPnP search > \- Allow user to manually enter Philips Hue local bridge IP address to configure it > > TODO: > \- Improve documentation (to explain that bridge should be registered through Philips Hue Application before using N-UPnP search) > \- Improve documentation to add the manual configuration > \- Ask for users to do real test use cases (Docker image under build) ---
**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:** [Novembre 10, 2023, 12:56 UTC](https://community.gladysassistant.com/t/philips-hue-ameliorations-du-scan/8427/10 "2023-11-10T12:56:30Z")
@cicoub13 Excellent boulot, c’est bon pour moi et c’est mergé sur master 👏 ---
**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:** [Novembre 20, 2023, 4:31 UTC](https://community.gladysassistant.com/t/philips-hue-ameliorations-du-scan/8427/11 "2023-11-20T16:31:51Z")
Cette fonctionnalité est disponible dans Gladys Assistant 4.31 ! 🚀 > [@Gladys Assistant 4.31 : Filtre par tag & MQTT dans les scènes](https://community.gladysassistant.com/t/gladys-assistant-4-31-filtre-par-tag-mqtt-dans-les-scenes/8511): > > Salut à tous ! Après Gladys Assistant 4.30 et sa fameuse alarme, voilà une nouvelle version de Gladys avec pas mal de fonctionnalités sympa slight_smile A découvrir ci-dessus ! Merci à @Lokkye et @cicoub13 pour leurs contributions clapclap Edit: Je profite de ce post pour dire que je recherche de l’aide ici pour écrire du contenu sur le blog Gladys ! Je ferme ce sujet, n’hésitez pas à créer un autre sujet en cas de bugs sur cette fonctionnalité 🙂 ---
**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:** [Novembre 20, 2023, 4:31 UTC](https://community.gladysassistant.com/t/philips-hue-ameliorations-du-scan/8427/12 "2023-11-20T16:31:53Z")