Effectivement @StephaneB , j’arrive à reproduire.
J’ai amélioré le fonctionnement du composant pour quelque chose de plus propre, et j’en ai profité pour rajouter le order_dir dans l’URL pour se souvenir de la direction du tri aussi
J’ai simplifié le code en faisant de l’URL la seule source de vérité, plus de state dupliqué qui du coup créait les bugs qu’on observait.
La PR :
GladysAssistant:master ← GladysAssistant:fix-tag-selection-move-order-dir-url
opened 02:29PM - 31 Jan 25 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] 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)
- [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.
### Description of change
Fix https://github.com/GladysAssistant/Gladys/issues/2218
Pour tester sur Gladys Plus :
Cool que tu aies pu identifier le souci et le corriger. Comme ce n’était pas systématique, j’avais peur que ce ne soit pas facile à reproduire pour toi…
1 « J'aime »
pierre-gilles:
La PR :
Corrigé dans Gladys Assistant 4.53.0 :
Salut à tous,
Les améliorations UX continuent d’affluer dans cette nouvelle version !
1 « J'aime »
