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
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
Salut @cicoub13
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)
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
Qui est ensuite passé en props au component : Gladys/front/src/routes/dashboard/index.js at master · GladysAssistant/Gladys · GitHub
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
Pour ça on trouvera quelqu’un qui en a un dès que la PR est prête !
Bonne nouvelle,
J’avais fait remonter le problème de détection il y a longtemps.
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
1 « J'aime »
@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 !
1 « J'aime »
Bien vu pour le tutoriel pour lancer une image Docker ! Merci d’avoir pris le temps
PR prête (testée par un utilisateur anglais)
GladysAssistant:master ← cicoub13:feat-philips-hue-improve-scan
opened 10:52AM - 30 Oct 23 UTC
### 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)
1 « J'aime »
@cicoub13 Excellent boulot, c’est bon pour moi et c’est mergé sur master
3 « J'aime »
Trop cool, merci pour la PR !
