(Kleine) Schreibweise auf der Seite „Integrationen“ V4.12.0

jparbel · 9. November 2022 um 20:02

Nur eine kleine Anmerkung zur Formulierung: (der Inhalt ist ansonsten ausgezeichnet!)
Text „Apple HomeKit“ sollte man lesen: „Verbinden Sie Gladys mit HomeKit und nutzen Sie Siri, um mit ihr zu sprechen“. anstelle von „…um mit ihr zu sprechen.“
gleiches gilt für die Seite /integration/communication
Gut gemacht an den Entwickler für diese Integration, auch wenn ich sie nicht testen kann.
Gute Nacht.

pierre-gilles · 11. November 2022 um 07:46

Hallo @jparbel Danke für dein Feedback, ich verstehe nicht, woher du dieses Problem siehst. Ist es in Gladys? Auf der Website? Hast du einen Link? Danke!

jparbel · 11. November 2022 um 09:37

Hallo @pierre-gilles,
Es geht um die Seiten „Alle Integrationen“ und „Kommunikation“:
« http://…/dashboard/integration » und
« http://…/dashboard/integration/communication »

Es sollte „sprechen“ und nicht „sprechen Sie“ heißen.
Guten Tag.

pierre-gilles · 11. November 2022 um 09:38

Ok! Danke für die Rückmeldung.

cc @bertrand Ein kleines Tippfehler bei der HomeKit-Integration

bertrandda · 11. November 2022 um 13:40

Danke @jparbel, der Fix ist da

github.com/GladysAssistant/Gladys

Fix typo HomeKit integration (#1649)

master ← bertrandda:fix-homekit-typo

ouvert 01:39PM - 11 Nov 22 UTC

bertrandda

+1 -1

### 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: - [ ] If your changes affects code, did your write the tests? - [ ] Are tests passing? (`npm test` on both front/server) - [ ] Is the linter passing? (`npm run eslint` on both front/server) - [ ] Did you run prettier? (`npm run prettier` on both front/server) - [ ] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front) - [ ] 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](https://community.gladysassistant.com/) for testing before merging. - [ ] 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 Please provide a description of the change here. It's always best with screenshots, so don't hesitate to add some!

pierre-gilles · 14. November 2022 um 10:00

Danke @bertrandda, es ist gemerged

Thema		Antworten	Aufrufe
Nouvelle vidéo : Gladys Assistant chez vous #5, Cyril et son café prêt tous les matins grâce à Gladys! Actualités	37	2692	10. Oktober 2022
Améliorations dans l'UI de l'intégration zigbee2mqtt Développement	20	609	1. November 2023
Pb synchronisation calendrier Configuration calendrier	20	332	3. April 2026
Correction diverses UX Développement ux	11	140	2. Juli 2026
Création d'appareil MQTT - amélioration sémantique Demande de fonctionnalités mqtt	15	499	8. Dezember 2023

(Kleine) Schreibweise auf der Seite „Integrationen“ V4.12.0

Verwandte Themen