Suggestion to delete entire timetable when user starts to delete the timetable#2134
Suggestion to delete entire timetable when user starts to delete the timetable#2134nilsreichardt merged 5 commits intomainfrom
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces a helpful feature that suggests deleting the entire timetable when a user deletes multiple lessons in a short time. It also adds the necessary backend logic to delete lessons in batches and includes a confirmation dialog with a countdown for safety. The implementation is solid, and the addition of tests is great. I have a couple of suggestions regarding a style guide violation and a potential bug in one of the test mocks.
|
Visit the preview URL for this PR (updated for commit d2a6bfd): https://sharezone-website-dev--pr2134-codex-delete-entire-k5z5tlm0.web.app (expires Tue, 10 Feb 2026 22:29:41 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 372b0431a96247f908d9a97d5d865de1c8b3b04e |
|
Visit the preview URL for this PR (updated for commit e3b4b99): https://sharezone-console-dev--pr2134-codex-delete-entire-o3vk6gy1.web.app (expires Tue, 10 Feb 2026 23:08:26 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 471536afe3f6ec4895d9ea75513730b515d17eb6 |
|
Visit the preview URL for this PR (updated for commit e3b4b99): https://sharezone-test--pr2134-codex-delete-entire-nlo37f7z.web.app (expires Tue, 10 Feb 2026 23:09:49 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 4cb3ae61e1e018abfd9841fd3239f5b49ccc034b |
DeleteAllLessons.mp4