Création d'appareil MQTT - amélioration sémantique

cce66 · November 9, 2023, 12:25pm

Bonjour

C’est une fonctionnalité purement esthétique mais qui me parait plus claire à comprendre pour ceux qui comme moi ont le cerveau lent et qui sont plus dans le vent .

Dans la création d’appareil MQTT je trouve qu’il serait plus compréhensif d’avoir
le 2d à la place du 1er

La version avec un bouton pour copier le topic serait encore plus top (là c’est plus de la sémantique je l’avoue ! )

spenceur · November 9, 2023, 5:23pm

Hello,

Il y a deux besoin sur une demande (vaut mieux créer une demande pour ton « bouton » copier)

Je comprend pas la valeur ajouté de la premiere demande perso surtout si cest juste pour le confort d’un ou deux users m’enfin .

Je pense que ce nest pas grand chose a faire, donc vaut mieux que tu fasses la modif et propose la pr

cce66 · November 10, 2023, 9:24am

Bonjour @spenceur,

Bah, je trouves que cela met plus en avant et en évidence le sens « Gladys vers l’extérieur » dans le cas que je propose ! Le premier je trouves me parait plus ambigu quant à la direction de l’écoute et la publication…

Je l’avoue je le ferais bien mais je sais pas trop comment faire si tu me pouvais me donner des pistes pour cela je suis preneur ! Cela pourrait me servir pour d’autres cas !

spenceur · November 10, 2023, 11:31am

github.com

GladysAssistant/Gladys/blob/master/front/src/config/i18n/fr.json#L1114


      
            "externalIdLabel": "ID externe de la fonctionnalité",
            "externalIdMessage": "L'ID externe d'une fonctionnalité est un ID unique qui vous aide à identifier cette fonctionnalité en dehors de Gladys lors de l'envoi d'un message MQTT. Il doit commencer par \"mqtt:\". Pour en savoir plus, rendez-vous sur la documentation de l'API MQTT de Gladys <a href=\"https://gladysassistant.com/fr/docs/api/mqtt-api\"> ici </a>.",
            "externalIdPlaceholder": "Message clé de la fonctionnalité MQTT",
            "minLabel": "Valeur minimum",
            "minPlaceholder": "Entrez la valeur minimum",
            "maxLabel": "Valeur maximum",
            "maxPlaceholder": "Entrez la valeur maximum",
            "readOnlyLabel": "Est-ce un capteur ?",
            "readOnlyButton": "Oui, ce capteur ne fait qu'envoyer des données à Gladys.",
            "addButton": "Ajouter une fonctionnalité",
            "mqttTopicToPublishExampleLabel": "Topic MQTT pour publier",
            "mqttTopicToPublishExampleDescription": "Gladys écoute ce topic. Si vous publiez une valeur dedans, Gladys l'associera à cette fonctionnalité. ( voir <a href=\"https://gladysassistant.com/fr/docs/api/mqtt-api/\" target=\"_blank\" rel=\"noopener noreferrer\">doc</a> )",
            "mqttTopicToListenExampleLabel": "Topic MQTT à écouter",
            "mqttTopicToListenExampleDescription": "Gladys publiera un message dans ce topic si cet appareil est contrôlé depuis Gladys. Vous devez écouter ce topic. ( voir <a href=\"https://gladysassistant.com/fr/docs/api/mqtt-api/\" target=\"_blank\" rel=\"noopener noreferrer\">doc</a> )",
            "copyMqttTopic": "Copier le topic MQTT",
            "copied": "Copié !",
            "copyFailed": "Erreur lors de la copie",
            "deleteLabel": "Supprimer fonctionnalité"
          },
          "setup": {
            "title": "Configuration MQTT",

github.com

GladysAssistant/Gladys/blob/master/front/src/config/i18n/fr.json#L1116


      
            "externalIdPlaceholder": "Message clé de la fonctionnalité MQTT",
            "minLabel": "Valeur minimum",
            "minPlaceholder": "Entrez la valeur minimum",
            "maxLabel": "Valeur maximum",
            "maxPlaceholder": "Entrez la valeur maximum",
            "readOnlyLabel": "Est-ce un capteur ?",
            "readOnlyButton": "Oui, ce capteur ne fait qu'envoyer des données à Gladys.",
            "addButton": "Ajouter une fonctionnalité",
            "mqttTopicToPublishExampleLabel": "Topic MQTT pour publier",
            "mqttTopicToPublishExampleDescription": "Gladys écoute ce topic. Si vous publiez une valeur dedans, Gladys l'associera à cette fonctionnalité. ( voir <a href=\"https://gladysassistant.com/fr/docs/api/mqtt-api/\" target=\"_blank\" rel=\"noopener noreferrer\">doc</a> )",
            "mqttTopicToListenExampleLabel": "Topic MQTT à écouter",
            "mqttTopicToListenExampleDescription": "Gladys publiera un message dans ce topic si cet appareil est contrôlé depuis Gladys. Vous devez écouter ce topic. ( voir <a href=\"https://gladysassistant.com/fr/docs/api/mqtt-api/\" target=\"_blank\" rel=\"noopener noreferrer\">doc</a> )",
            "copyMqttTopic": "Copier le topic MQTT",
            "copied": "Copié !",
            "copyFailed": "Erreur lors de la copie",
            "deleteLabel": "Supprimer fonctionnalité"
          },
          "setup": {
            "title": "Configuration MQTT",
            "mqttDescription": "Vous pouvez connecter Gladys à un broker (serveur) MQTT pour envoyer et recevoir des messages MQTT. En savoir plus sur l'API Gladys MQTT <a href=\"https://gladysassistant.com/fr/docs/api/mqtt-api/\">ici</a>.",
            "embeddedBrokerLabel": "Broker dans un conteneur Docker",

cce66 · November 10, 2023, 4:28pm

Merci @spenceur mais après il faut faire comment ? Je suis débutant dans ce domaine sur github…

Tlse-vins · November 10, 2023, 4:33pm

Regarde de tutoriel tu devrais y arriver.

cce66 · November 10, 2023, 5:17pm

merci @Tlse-vins je vais regarder cela !

pierre-gilles · November 30, 2023, 9:00am

Impossible d’utiliser l’API de copier/coller sur un site non HTTPS. Possible sur Gladys Plus mais pas en local, donc cette fonctionnalité a été retiré (elle existait dans le passé). On pourrait la remettre sur Gladys Plus

Pour les changements sur les titres, c’est pas flagrant pour moi le changement. Le texte en dessous explique déjà bien ce que ça fait non ?

cce66 · November 30, 2023, 10:43am

dommage pour le copier-coller

Oui mais c’est le sens implicite qui manques je trouves, peut-être ajouter le sens pour que cela soit instantanément plus clair ?
Topic MQTT pour publier => Topic MQTT pour publier vers Gladys
Topic MQTT à écouter => Topic MQTT à écouter de Gladys
Dans l’état actuel, j’ai souvent tendance à copier le mauvais topic
Je ne sais pas si je suis le seul dans ce cas…

pierre-gilles · November 30, 2023, 10:44am

OK je comprend, pas de souci pour rendre ça plus explicite

pierre-gilles · November 30, 2023, 12:57pm

@cce66 Je t’ai fais des retours sur ta PR :

github.com/GladysAssistant/Gladys

Update fr.json

GladysAssistant:master ← cce66:patch-1

opened 11:40PM - 28 Nov 23 UTC

cce66

+40 -40

corrections typo et proposition changement libellé sur les appareils MQTT ###… Pull Request check-list To ensure your Pull Request can be accepted as fast 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 ([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 Please provide a description of the change here. It's always best with screenshots, so don't hesitate to add some!