bab85
Décembre 19, 2023, 7:53
1
Suite du sujet Amélioration des tags :
Bonsoir tout le monde,
Pour faire suite aux citations ci-dessus, j’ai pu constater qu’un tag qui n’est affecté qu’à des scènes qui sont désactivées est supprimé.
Est-ce volontaire ou fortuit ?
Etant toujours en phase d’apprentissage et de test, j’ai du matériel qui sert à plusieurs scènes, donc j’active ou désactive régulièrement des scènes. Mais si un tag ne se trouve plus sur une scène active, il est supprimé. Je comprends qu’en vitesse de croisière ce problème ne se pose sûrement pas, donc si c’est volontaire, je ferai avec (nous n’avions pas accès au tag il y a encore peu ), mais si ça se fait comme ça sans plus de raison, je me permettrai de faire une demande de fonctionnalité .
Psoy
Décembre 20, 2023, 6:55
2
bonjour @bab85
Oui ce bugs est deja signalé, on en parle à partir de là :
pour les dev
quelques remarques
[image]
Il faudrait un petit texte pour préciser qu’il faut taper le nom du tag dans la zone « Select » ou remplacer « Select » par quelque chose comme « Entrez le nom du tag ici » car pas évident à trouver la première fois
Un petit bug
j’ai mes scènes
[image]
j’actives les tags
[image]
je coche le filtrage se fait bien
[image]
je décoche mais les scènes ne réapparaissent pas
[image]
il faut cliquer sur plans puis à nouveau sur Scènes p…
Mais je pensais que c’etait reglé avec la version 4.32
bab85
Décembre 20, 2023, 8:04
3
Bonjour @Psoy
Effectivement je n’avais pas fait attention qu’il en était déjà fait mention (en fait, dans le fil de discussion, je n’avais pas compris que le tag était supprimé).
Par contre je suis en v4.33 et je te confirme que ce fonctionnement est toujours présent.
Yes c’est volontaire Un tag n’existe que dans une scène, donc si aucune scène n’a ce tag, il n’existe pas
bab85
Janvier 6, 2024, 11:05
5
Mes meilleurs voeux à tous pour cette nouvelle année.
@pierre-gilles mes tags qui disparaissent sont bien associées à des scènes, mais ce sont des scènes désactivées.
Oui en effet, idem pour moi aujourd’hui… ce n’est pas le comportement voulu. A voir !
Ok, dans ce cas je veux bien que quelqu’un créé une issue Github pour garder une trace du bug
Salut @pierre-gilles ,
Je t’ai repondu, tu as visiblement oublié que le sujet a déjà été évoqué 2 fois. J avais pourtant mis la référence à cette discussion dans l’issue :
ouvert 01:02PM - 08 Jan 24 UTC
Related to the subject: [Add any other context about the problem here.](https://… community.gladysassistant.com/t/suppression-des-tags-des-scenes-desactivees/8642)
**Describe the bug**
When all scenes of a tag are only disabled, the tag is deleted as if no scene contained this tag anymore even though it is just disabled
**To Reproduce**
Steps to reproduce the behavior:
- Go to scenes
- Create a “Test tag scene deactivated” scene
- Add a “Disabled Scenes” tag
- Save and return to the Scenes list menu
- Select the new tag "Deactivated scenes" to find the newly created scene
- Disable scene
- Refresh the page
- Search again for the “Disabled Scenes” tag => It no longer exists
**Expected behavior**
The tag should always exist when at least one scene, disabled or not, has this tag.
Ah pardon j’ai lu un peu vite, c’est bien un bug dans ce cas là
1 « J'aime »
J’ai regardé ce bug, c’est vraiment tout bête !
C’est corrigé ici :
GladysAssistant:master
← GladysAssistant:fix-tags-bug
opened 06:17PM - 19 Apr 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
Fix https://github.com/GladysAssistant/Gladys/issues/2000
2 « J'aime »