bonjour , juste une petite suggestion car je me suis fait avoir, l’ors d’un ajout dans le dash , j’ai glissé et cliqué sur supprimer, tout mon dash a été supprimé sans avertissement j’ai du tout reconfigurer, peut on mettre un message qui nous prévient de cette suppression, je sais pas si d’autres ce sont fait avoir ?
Psoy
Février 22, 2022, 9:17
2
Et oui moi aussi j’y ai eu droit, donc je vote
qleg
Janvier 8, 2023, 3:05
3
Hello, je viens de me faire avoir également
1 « J'aime »
C’est les débuts c’est pour ça.@qleg
1 « J'aime »
Lokkye
Janvier 21, 2023, 8:17
5
Petite proposition pour la confirmation de suppression :
Est ce que cela vous conviendrait ?
Si oui voici la PR :
GladysAssistant:master ← callemand:validate-delete-dashbord
opened 08:17PM - 21 Jan 23 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)
- [ ] 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](https://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
Just adding validation before delete dashboard
5 « J'aime »
C’est parfait !
Merci pour la PR !
J’ai fais une review, il y a des petites fautes d’orthographes dans les traductions, je te laisse corriger et ensuite c’est bon pour moi
Lokkye
Janvier 23, 2023, 7:40
7
@pierre-gilles : C’est corrigé.
Je me disais que je pouvais aussi faire la même chose au niveau des scènes. Si c’est ok pour toi, je ferais une autre PR.
2 « J'aime »
Merci !
Yes effectivement, go ! Merci !
Jluc
Janvier 24, 2023, 10:24
9
Ce serait bien, en supprimant une scène issue d’une duplication, j’ai dû aller trop vite et perdu les deux
@Lokkye Merci pour ta PR, c’est mergé sur master !
Ca partira dans la prochaine version de Gladys.
Je ferme cette demande pour relâcher les votes
4 « J'aime »
: Voté
