Phil
January 5, 2023, 7:14am
1
Hello,
I tested the « database cleanup » option…The cleanup has started… Gladys will be unavailable until the cleanup finishes. »
Wouldn’t it be useful to have:
a start date/time of the cleanup added to the message « the cleanup has started .. »
a message « date/time: cleanup finished, Gladys is available »
Phil
Hi @Phil
Thanks for this request, you’re right!
It’s fixed in this PR:
master ← add-websocket-event-vacuum-finished
ouvert 01:52PM - 10 Feb 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://community.gladysassistant.com/t/fin-de-nettoyage/7819
From now on, at the end of the cleanup, a message is displayed:
Now that sensor values are no longer in SQLite but in DuckDB, cleanups should be very fast
2 Likes
Fixed in Gladys Assistant 4.53.0:
Salut à tous,
Les améliorations UX continuent d’affluer dans cette nouvelle version !
