Menüs einheitlich gestalten

Tlse-vins · 3. Dezember 2023 um 16:34

Ich habe bemerkt, dass die Menüs in den Integrationen nicht alle identisch sind. Bei einigen Integrationen fehlt sogar der Link zur Dokumentation.

– Tuya-Integration:
Tuya-Dokumentation statt Tuya Dokumentation

– TP-Link-Integration
Fehlender Link zur Dokumentation TP-Link-Dokumentation

– Xiaomi Home
Fehlender Link zur Dokumentation Xiaomi-Dokumentation

– Telegram
Es gibt einen Link im Text, aber keinen Button Telegram-Dokumentation

– Philips Hue
Fehlender Link zur Dokumentation Philips Hue-Dokumentation PR erstellt

– Owntracks
Es gibt einen Link im Text, aber keinen Button Owntracks-Dokumentation

– Nextcloud Nextcloud talk
Fehlender Link zur Dokumentation Nextcloud talk-Dokumentation

– MQTT
Fehlender Link zur Dokumentation MQTT-Dokumentation

– Melcloud
Fehlender Link zur Dokumentation Melcloud-Dokumentation

– LAN Manager
Fehlender Link zur Dokumentation LAN Manager-Dokumentation

– Google Home
Es gibt einen Link im Text, aber keinen Button Google Home-Dokumentation

– eWelink
Fehlender Link zur Dokumentation eWelink-Dokumentation

– Enedis
Fehlender Link zur Dokumentation Enedis-Dokumentation

– Kamera
Fehlender Link zur Dokumentation Kamera-Dokumentation

– CalDAV
Fehlender Link zur Dokumentation CalDAV-Dokumentation

– Broadlink
Fehlender Link zur Dokumentation Broadlink-Dokumentation

– Bluetooth
Fehlender Link zur Dokumentation Bluetooth-Dokumentation

– Apple HomeKit
Fehlender Link zur Dokumentation Apple HomeKit-Dokumentation

– OpenWeather
Fehlender Link zur Dokumentation OpenWeather-Dokumentation

– Amazon Alexa
Es gibt einen Link im Text, aber keinen Button Amazon Alexa-Dokumentation

pierre-gilles · 4. Dezember 2023 um 18:01

Warum nicht Jeder Pull-Request ist willkommen, um das zu ändern!

Tlse-vins · 4. Dezember 2023 um 18:18

Ich kümmere mich um die Dokumentationen auf der Website, kann sich jemand darum kümmern?

Ich habe mich noch nicht mit den PRs für Gladys beschäftigt. Ich weiß nicht genau, wo ich die Änderungen hinzufügen soll.

@Terdious könntest du mich vielleicht führen, falls nötig?

Aber nicht sofort, zuerst die Dokumentationen.

Brisou · 4. Dezember 2023 um 18:25

Hallo @Tlse-vins,

Ich schaue mir das gerne an und sage dir dann, wie ich es gemacht habe, wenn du mehr Zeit hast.

Tlse-vins · 4. Dezember 2023 um 18:32

Alles klar !

Brisou · 6. Dezember 2023 um 10:24

Hallo,

Hier ein paar Screenshots:

@Tlse-vins
Entspricht das dem, was du im Sinn hattest?
Ich habe versucht, das Design zu vereinheitlichen.

Für die Integrationen NextCloud, MELCloud, LAN Manager, eWiLink und ChatGPT gibt es keine Dokumentation auf der Website, möchtest du trotzdem einen Link?

In der Integration Caméra gibt es bereits einen Dokumentationsbutton, möchtest du diesen auch ersetzen?

Ich habe auch bemerkt, dass die Icons manchmal inkonsistent sind (z. B. im Geräte- oder Konfigurations-Tab), gibt es irgendwo eine „Grafikrichtlinie“?

@pierre-gilles
Übrigens, ich habe ein kleines Problem mit den Alexa- und Google Home-Seiten, falls jemand mir helfen kann?

Tlse-vins · 6. Dezember 2023 um 12:31

eWiLink ist die Sonoff-Integration
LAN Manager muss ich machen
NextCloud muss ich mir ansehen
MELCloud muss ich mir ansehen
ChatGPT gibt es nichts zu konfigurieren

Bei den Icons bin ich einverstanden, ich hatte sie nicht gesehen. Der von Tuya gefällt mir ganz gut.

pierre-gilles · 6. Dezember 2023 um 17:08

Ja? Keine Erlaubnis nötig, aha, einfach fragen

Brisou · 6. Dezember 2023 um 18:07

Ich kann die Variable user (oder eine prop) nicht in die Seite welcome.jsx für Alexa oder Google Home hochladen, um den Link zur Dokumentation anzeigen zu können.

Tlse-vins · 6. Dezember 2023 um 21:03

MELCloud ist von @lokkye, wer es erstellt hat.
@lokkye, kannst du die Dokumentation erstellen?

NextCloud ist von @bertrandda, ebenfalls, kannst du die Dokumentation erstellen?
LAN Manager ist von @alextrovato, ebenfalls, kannst du die Dokumentation erstellen?

pierre-gilles · 7. Dezember 2023 um 08:27

Hast du einen connect um dein Component herum gesetzt, um den Benutzer zu injizieren?

Beispiel:

github.com/GladysAssistant/Gladys

front/src/routes/dashboard/index.js

master


      
              const errorMessageOtherFormat = get(e, 'response.data.message');
              if (status === 401 && errorMessageOtherFormat === 'TABLET_IS_LOCKED') {
                this.redirectToLocked();
              }
            }
          };
          
          alarmArming = () => {};
          
          constructor(props) {
            super(props);
            this.props = props;
            // per-selector request generations for fetchDashboardConfig — plain
            // instance state, nothing renders from it
            this.dashboardConfigGenerations = new Map();
            this.state = {
              isGladysPlus: this.props.session.gatewayClient !== undefined,
              dashboardDropdownOpened: false,
              defineTabletModeOpened: false,
              dashboardEditMode: false,
              showReorderDashboard: false,

Brisou · 7. Dezember 2023 um 10:09

Ich habe den Benutzer im Connect verwendet.

export default connect(‹ user,session ›, {})(GoogleHomeGateway);

Was ich versucht habe:

Überprüfen, ob der Benutzer im Connect ist.
Die notwendigen Imports einfügen.
{…props} und user={props.user} durchgeben.
Ein <GoogleWelcomePage {…props} /> erstellen.
Ein erstellen.

Ich suche weiter

EDIT: Daran hatte ich nicht gedacht, aber deine Idee, ein Connect in der Komponente zu verwenden, funktioniert!!!
Ich wusste nicht, dass man es direkt in der Komponente verwenden kann und nicht nur in index.js
Vielen Dank!

pierre-gilles · 7. Dezember 2023 um 12:27

Ansonsten reicht es, den Benutzer von index.js aus als Props zu übergeben.

Genau so, hat das nicht funktioniert?

Brisou · 7. Dezember 2023 um 13:12

Ich habe es auf verschiedene Arten versucht, aber es hat nicht funktioniert.

Auch nicht…

Brisou · 8. Dezember 2023 um 08:09

PR gesendet, hier ist der Link:

github.com/GladysAssistant/Gladys

feat: Add doc link on each integration + style consistency (#1969)

master ← jbrisavoine:front-menu-homogene

ouvert 03:30PM - 07 Dec 23 UTC

jbrisavoine

+996 -630

### 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? - [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) - [ ] 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. - [ ] 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 the request [https://community.gladysassistant.com/t/rendre-les-menus-homogene/8581/1](url): - Add documentation link on each integration pages. - Change icons and style for consistency - Update `src/components/documentation/DeviceConfigurationLink.jsx` to be able to chose the path of the documentation Before: <img width="931" alt="2023 12 07_PR_BEFORE" src="https://github.com/GladysAssistant/Gladys/assets/138247436/15d0e725-6f1f-4f5a-9fc0-b0a48762a9c4"> After: <img width="941" alt="2023 12 07_PR_AFTER" src="https://github.com/GladysAssistant/Gladys/assets/138247436/ed223cca-dfb8-49bb-a442-70ee84c53b78">

pierre-gilles · 8. Dezember 2023 um 09:02

Danke für den PR @Brisou, das ist sehr schön und funktioniert super gut

Ich habe dir ein paar Rückmeldungen zur PR gegeben:

github.com/GladysAssistant/Gladys

feat: Add doc link on each integration + style consistency (#1969#pullrequestreview-1771834650)

master ← jbrisavoine:front-menu-homogene

ouvert 03:30PM - 07 Dec 23 UTC

jbrisavoine

+996 -630

### 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? - [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) - [ ] 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. - [ ] 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 the request [https://community.gladysassistant.com/t/rendre-les-menus-homogene/8581/1](url): - Add documentation link on each integration pages. - Change icons and style for consistency - Update `src/components/documentation/DeviceConfigurationLink.jsx` to be able to chose the path of the documentation Before: <img width="931" alt="2023 12 07_PR_BEFORE" src="https://github.com/GladysAssistant/Gladys/assets/138247436/15d0e725-6f1f-4f5a-9fc0-b0a48762a9c4"> After: <img width="941" alt="2023 12 07_PR_AFTER" src="https://github.com/GladysAssistant/Gladys/assets/138247436/ed223cca-dfb8-49bb-a442-70ee84c53b78">

(Nichts Ernstes)

Brisou · 8. Dezember 2023 um 09:09

@pierre-gilles alles klar, danke

Brisou · 8. Dezember 2023 um 10:31

@pierre-gilles ich habe den Code geändert

pierre-gilles · 8. Dezember 2023 um 10:33

Danke! Du musst die von der PR geänderten Dateien entfernen, die nicht zur PR gehören (cypress, package.json, etc.).

Brisou · 8. Dezember 2023 um 10:34

Ich würde ja gerne, aber wie soll ich das machen?

Thema		Antworten	Aufrufe
Gladys Assistant 4.36 : Intégration Netatmo 🔥 Actualités	18	726	12. Februar 2024
📈 Passage à Node.js LTS 16/18 + fin de Open-Zwave Développement	11	1027	2. Dezember 2022
Création d'appareil MQTT - amélioration sémantique Demande de fonctionnalités mqtt	15	499	8. Dezember 2023
Ajout intégration sonos - notamment enceintes Demandes livrées feature-integration , sonos	32	2727	11. Dezember 2023
Gladys Assistant: Ecrivons plus de contenu ensemble! Développement	65	1449	17. Mai 2024

Menüs einheitlich gestalten

Verwandte Themen