J’ai un abonnement Heures Creuses Soir et Week-End et la donnée des HC Week-End ne remonte pas dans Gladys.
https://community.gladysassistant.com/t/lixee-tic-et-engie-ou-autre-mais-pas-enedis/8707/20?u=gaetanb76
@pierre-gilles ,
J’ai réalisé une PR pour ajouter cette data EASF03.
GladysAssistant:master
← William-De71:add-new-func-lixee
opened 08:56PM - 13 Oct 24 UTC
### 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 ([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!
J’en ai profité aussi pour corriger l’affichage de l’icone et texte pour l’expose RELAIS car rien ne s’affichait.
Par contre je me pose une question si on pouvait pas améliorer l’affichage des fonctionnalités, car par exemple il y a beaucoup d’index, intensité, puissance mais on ne sait pas vraiment à quoi cela correspond! Sans ouvrir la page zigbee2mqtt impossible de savoir qui est qui…
Voir ci-dessous.
Voilà ce que je voudrais faire:
2 « J'aime »
Merci pour la PR @_Will_71 , c’est mergé et ça partira dans la prochaine version de Gladys
ça serait super sympa oui ! Tu voudrais t’y prendre comment ? Faire des features spécifiques au Lixee TIC ?
Dans mon idée oui, car avec du générique quand on se retrouve avec plusieurs fonctionnalités identiques je trouve cela compliqué a savoir qui est qui
Après si tu as d’autres idées je suis preneur.
Non ça me semble bien de créer des features très spécifiques dans certains cas comme ici, au moins on peut offrir une expérience plus précise
je proposerais quelque chose mais là où je serais moins sûr c’est les traductions en allemand.
j’utiliserai des outils en lignes pour traduire mais j’ai pas encore une confiance absolu aux traducteurs
ChatGPT @_Will_71 pour la traduction, beaucoup plus performant que google trad ou autre.
Il me semble que Pierre-Gilles l’utilise également.
2 « J'aime »
Merci du conseil @Terdious ,
1 « J'aime »
Oui, moi je prend le JSON en français, je le colle, et je demande de traduire en anglais et allemand :
ça prend 10 secondes
2 « J'aime »
Ok je ferais comme cela merci
Je viens d’essayer et c’est clair que c’est efficace. Merci du conseil!
2 « J'aime »
@pierre-gilles la PR avec les features spécifiques
GladysAssistant:master
← William-De71:lixee-tic
opened 09:25PM - 17 Oct 24 UTC
### 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:
- [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)
- [x] 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.
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!
![image](https://github.com/user-attachments/assets/f045172f-a9de-4991-afcb-50b84a548522)
![image](https://github.com/user-attachments/assets/a257c186-992e-45a4-816f-18f3d2662d7a)
1 « J'aime »
Merci @_Will_71 ! Beau travail Tu as testé ?
Et c’est rétrocompatible ? Si quelqu’un lance cette version de Gladys alors qu’il a déjà un Lixee, ça pète ou ça marche ?
J’ai fait les tests en réel avec mon LIxee TIC mais je l’ai ajouté sur ma nouvelle instance pour le test.
Je referais un test avec le device déjà intégré dans Gladys mais je pense que cela va tout casser car je change les fonctionnalités du Lixee
@pierre-gilles , Je viens de faire un essai et effectivement cela casse un peu tout. Il reste uniquement la feature sur de l’intensité du signal que j’ai pas modifié.
J’ai vu des fichiers de migration, je pense qu’il faut que je fasse cela pour proposer une migration des anciennes vers les nouvelles features.
Je regarderais dans le week-end ce que je peux faire.
Edit: cela casse tout, non c’est juste l’affichage sur le dashboard
1 « J'aime »
@_Will_71 Je ferme ce sujet car la donnée EASF03 a bien été ajoutée dans Gladys Assistant 4.47.0 :
Salut à tous !
Je suis bien rentré du Japon en début de semaine, et je publie aujourd’hui les développements des deux dernières semaines
DuckDB 1.1.1 : DuckDB a publié des mises à jour pour corriger des bugs identifiés depuis leur version 1.0. Chez Gladys, un de ces bugs empêchait certains utilisateurs de réimporter leurs sauvegardes Gladys Plus. Bien que je n’aie pas réussi à reproduire ce problème de restauration, je pense que la mise à jour en version 1.1.1 devrait le résoudre. J’a…
On en parle sur un autre sujet dans Développement ?