Rendre les menus homogène

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

4 Likes

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

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.

Hello @Tlse-vins,

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

Ok super !

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 ?

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.

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

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.

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 ?

1 Like

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

Exemple:

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 :nerd_face:

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 :upside_down_face:
Merci beaucoup !

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

Exactement comme ça, ça ne marchait pas?

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

Non plus…

PR envoyé, voici le lien :

1 Like

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

Je t’ai fais quelques retours sur la PR :

(Rien de très sérieux)

@pierre-gilles je check, merci

1 Like

@pierre-gilles j’ai modifié le code

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

Je veux bien mais comment faire ?