Rendre les menus homogène

Tlse-vins · Décembre 3, 2023, 4:34

J’ai remarqué dans les intégrations que les menus n’étaient pas tous identiques il manque même sur certaine intégration le lien vers la documentation.

– Intégration Tuya :
Documentation Tuya au lieu de Tuya documentation

– Integration TP-Link
Manque le lien pour la documentation Documentation TP-Link

– Xiaomi Home
Manque le lien pour la documentation Documentation Xiaomi

– Telegrame
Il y a un lien dans le texte mais pas de bouton Documentation Telegram

–Philips Hue
Manque le lien pour la documentation Documentation Philips Hue PR rédigé

– Owntracks
Il y a un lien dans le texte mais pas de bouton Documentation Owntracks

– Nextcloud Nextcloud talk
Manque le lien pour la documentation Documentation Nextcloud talk

– MQTT
Manque le lien pour la documentation Documentation MQTT

– Melcloud
Manque le lien pour la documentation Documentation Melcloud

– LAN Manager
Manque le lien pour la documentation Documentation LAN Manager

– Google Home
Il y a un lien dans le texte mais pas de bouton Documentation Google Home

– eWelink
Manque le lien pour la documentation Documentation eWelink

– Enedis
Manque le lien pour la documentation Documentation Enedis

– Caméra
Manque le lien pour la documentation Documentation Caméra

– CalDAV
Manque le lien pour la documentation Documentation CalDAV

– Broadlink
Manque le lien pour la documentation Documentation Broadlink

– Bluetooth
Manque le lien pour la documentation Documentation Bluetooth

– Apple HomeKit
Manque le lien pour la documentation Documentation Apple homeKit

– OpenWeather
Manque le lien pour la documentation Documentation OpenWeather

– Amazon Alexa
Il y a un lien dans le texte mais pas de bouton Documentation Amazon Alexa

pierre-gilles · Décembre 4, 2023, 6:01

Pourquoi pas Toute PR est la bienvenue pour changer ça !

Tlse-vins · Décembre 4, 2023, 6:18

Je m’occupe des documentations sur le site, est ce quelqu’un peut s’en occuper ?

Je ne me suis pas pencher sur les PR pour Gladys pour le moment. Je ne sais pas exactement où rajouter les modifications.

@Terdious tu pourrais me guider si jamais ?

Mais pas de suite, d’abord les documentations.

Brisou · Décembre 4, 2023, 6:25

Hello @Tlse-vins,

Je veux bien regarder et ensuite te dire comment j’ai fait quand tu auras plus de temps.

Tlse-vins · Décembre 4, 2023, 6:32

Ok super !

Brisou · Décembre 6, 2023, 10:24

Hello,

Ci-dessous quelques captures :

@Tlse-vins
Est-ce que cela correspond à ce que tu avais en tête ?
J’ai essayé d’unifier le rendu.

Pour les intégrations NextCloud, MELCloud, LAN Manager, eWiLink et ChatGPT, il n’y a pas de documentation sur le site, veux-tu quand même un lien ?

Dans l’intégration Caméra, il existe déjà un bouton Documentation, veux-tu aussi le remplacer ?

J’ai aussi remarqué que parfois les icônes étaient incohérentes (pour l’onglet appareils ou configuration), avons-nous une « charte graphique » quelque part ?

@pierre-gilles
Autrement, j’ai un petit souci avec les pages Alexa et Google Home, si jamais quelqu’un peut m’aider ?

Tlse-vins · Décembre 6, 2023, 12:31

eWiLink c’est l’intégration Sonoff
LAN Manager il faut que je la fasse
NextCloud il faut que je regarde
MELCloudil faut que je regarde
ChatGPT, il n’y a rien à configurer

Pour les icones, je suis d’accord, je n’avais pas vue. Celui de Tuya est pas mal à mon avis.

pierre-gilles · Décembre 6, 2023, 5:08

Oui ? Pas besoin de demander la permission aha, demande juste

Brisou · Décembre 6, 2023, 6:07

Je n’arrive pas à faire remonter la variable user (ou bien une prop) dans la page welcome.jsx pour Alexa ou bien Google Home afin de pouvoir faire apparaitre le lien de la doc.

Tlse-vins · Décembre 6, 2023, 9:03

MELCloud c’est @lokkye, qui l’a créé.
@lokkye, est ce que que tu peux faire la documentation ?

NextCloud c’est @bertrandda, idem, est ce que tu peux faire la documentation ?
LAN Manager c’est @alextrovato, idem, est ce que tu peux faire la documentation ?

pierre-gilles · Décembre 7, 2023, 8:27

Est-ce que tu as mis un connect autour de ton component pour injecter le user ?

Exemple:

github.com

GladysAssistant/Gladys/blob/master/front/src/routes/dashboard/index.js#L274


      
                  toggleFullScreen={this.toggleFullScreen}
                  toggleDefineTabletMode={this.toggleDefineTabletMode}
                  fullScreen={props.fullScreen}
                  hideExitFullScreenButton={props.fullscreen === 'force'}
                  isGladysPlus={isGladysPlus}
                />
              );
            }
          }
          
          export default connect(
            'user,session,fullScreen,currentUrl,httpClient,gatewayAccountExpired,tabletMode',
            actions
          )(Dashboard);

Brisou · Décembre 7, 2023, 10:09

J’ai bien user dans le connect.

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

Ce que j’ai essayé :

Vérifier si user est bien dans le connect.
Insérer les imports qu’il faut.
Essayer de passer {…props} et user={props.user}.
Créer un <GoogleWelcomePage {…props} />.
Créer un < GoogleWelcomePage user={props.user} />.

Je continue à chercher

EDIT : je n’y avais pas pensé, mais ton idée de mettre un connect dans le component fonctionne !!!
je ne savais pas qu’on pouvait le mettre directement dans le component et non pas que dans index.js
Merci beaucoup !

pierre-gilles · Décembre 7, 2023, 12:27

Sinon, il suffit de passer le user en props depuis le index.js

Exactement comme ça, ça ne marchait pas?

Brisou · Décembre 7, 2023, 1:12

J’ai essayé de plusieurs façons, mais ça ne fonctionnait pas.

Non plus…

Brisou · Décembre 8, 2023, 8:09

PR envoyé, voici le lien :

github.com/GladysAssistant/Gladys

feat: Add doc link on each integration + style consistency

GladysAssistant:master ← jbrisavoine:front-menu-homogene

opened 03:30PM - 07 Dec 23 UTC

jbrisavoine

+1666 -871

### 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 · Décembre 8, 2023, 9:02

Merci pour la PR @Brisou, c’est très chouette et ça marche super bien

Je t’ai fais quelques retours sur la PR :

github.com/GladysAssistant/Gladys

feat: Add doc link on each integration + style consistency

GladysAssistant:master ← jbrisavoine:front-menu-homogene

opened 03:30PM - 07 Dec 23 UTC

jbrisavoine

+1666 -871

### 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">

(Rien de très sérieux)

Brisou · Décembre 8, 2023, 9:09

@pierre-gilles je check, merci

Brisou · Décembre 8, 2023, 10:31

@pierre-gilles j’ai modifié le code

pierre-gilles · Décembre 8, 2023, 10:33

Merci ! Il faut que tu retire les fichiers modifiés par la PR pas lié à la PR (cypress, package.json, etc…)

Brisou · Décembre 8, 2023, 10:34

Je veux bien mais comment faire ?