Hallo,
Ich habe mit der Verbesserung des Philips Hue-Scans begonnen:
Hinzufügen des N-UPnP-Scans (dazu ist jedoch eine zusätzliche Anfrage erforderlich, um die Seriennummer abzurufen)
Möglichkeit, eine IP-Adresse manuell über die Frontend-Oberfläche einzugeben
Ich brauche Hilfe bei drei Punkten:
das Design des Frontend-Formulars. Ich habe es in einer Karte und in einem Formular versucht, aber es sieht immer schrecklich aus
ich kann den Wert bridge.ipaddress nicht aus einem Eingabefeld setzen und später verwenden. Das sollte eigentlich einfach sein, aber ich komme nicht weiter (in SetupTab.jsx)
ich kann nicht wirklich testen, weil ich keine Philips Hue-Brücke mehr habe
Der Branch ist hier: feat(philips-hue): N-UPnP Search + manual configuration from IP · cicoub13/Gladys@df3fe52 · GitHub
Ah ja, in der Dokumentation steht, dass der N-UPnP-Scan nur funktioniert, wenn die Bridge in der offiziellen App registriert wurde. Das müssen wir als Voraussetzung in unsere Dokumentation aufnehmen.
Hallo @cicoub13 Super cool, danke für das PR!
Vielleicht könntest du das Format übernehmen, das wir bei den anderen Integrationen haben?
Ein « + » Button neben « Netzwerksuche », der eine Karte hinzufügt (lass dich vom Design der anderen Gerätekarten inspirieren)
Ich würde dir empfehlen, dir Code von anderen Stellen in Gladys anzuschauen, um es zu verstehen
Beispiel für die Dashboard-Seite, wir erstellen eine Funktion: Gladys/front/src/routes/dashboard/index.js at master · GladysAssistant/Gladys · GitHub
Die dann als Props an das Component weitergegeben wird: Gladys/front/src/routes/dashboard/index.js at master · GladysAssistant/Gladys · GitHub
Falls du Anfänger mit Preact/React bist, könnte ein Tutorial dir vielleicht helfen?
Die React-Dokumentation ist super gut gemacht und es gibt ein interaktives Tutorial: Quick Start – React
Dafür werden wir jemanden finden, der eine hat, sobald das PR fertig ist!
Gute Nachricht,
Ich hatte das Problem mit der Erkennung schon vor langer Zeit gemeldet.
Bei manchen funktionierte es gut, bei anderen überhaupt nicht.
Front und Back funktionieren
Ein Image ist zum Testen verfügbar:
Scannen und Hinzufügen funktioniert weiterhin
Scan N-UPnP funktioniert für diejenigen, die zuvor Probleme hatten
Manuelle Hinzufügung über IP funktioniert
Steuerung von Geräten (Lampen) funktioniert weiterhin (da ich die Speicherung der Bridge-Konfigurationen leicht angepasst habe)
docker pull cicoub13/gladys:philips-hue-improve-scan
@cicoub13 Super cool, dass du das Problem im Frontend gefunden hast
Limite, poste doch eine Nachricht im Forum auf Englisch, vielleicht kann der Benutzer, der das Problem hatte, auch testen!
Gut gesehen für das Tutorial zum Starten eines Docker-Images! Danke, dass du dir die Zeit genommen hast
PR bereit (getestet von einem englischen Nutzer)
master ← cicoub13:feat-philips-hue-improve-scan
ouvert 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)
@cicoub13 Großartige Arbeit, das ist für mich in Ordnung und es ist in master gemerged