fixup! fixup! editoast: work-schedules: add endpoints for easier edition and viewing

Update editoast/src/views/work_schedules.rs

Co-authored-by: Léo Valais <[email protected]>

Author: eckter

editoast/src/views/work_schedules.rs

@@ -367,34 +367,14 @@ async fn delete_group(
 
     let conn = &mut db_pool.get().await?;
 
-    conn.transaction(|conn| {
-        Box::pin(async move {
-            // Check that the group exists
-            let work_schedule_group =
-                WorkScheduleGroup::retrieve_or_fail(&mut conn.clone(), group_id, || {
-                    WorkScheduleError::WorkScheduleGroupNotFound { id: group_id }
-                })
-                .await?;
-
-            // List work schedules to be deleted
-            let selection_setting = SelectionSettings::new()
-                .filter(move || WorkSchedule::WORK_SCHEDULE_GROUP_ID.eq(group_id));
-            let work_schedule_ids = WorkSchedule::list(&mut conn.clone(), selection_setting)
-                .await?
-                .into_iter()
-                .map(|work_schedule| work_schedule.id);
-
-            // Delete them
-            let conn = &mut db_pool.get().await?;
-            WorkSchedule::delete_batch(conn, work_schedule_ids).await?;
-
-            // Delete the group itself
-            WorkScheduleGroup::delete(&work_schedule_group, conn).await?;
-
-            Ok(axum::http::StatusCode::NO_CONTENT)
+    let work_schedule_group =
+        WorkScheduleGroup::retrieve_or_fail(&mut conn.clone(), group_id, || {
+            WorkScheduleError::WorkScheduleGroupNotFound { id: group_id }
         })
-    })
-    .await
+            .await?;
+    WorkScheduleGroup::delete(&work_schedule_group, conn).await?;
+
+    Ok(axum::http::StatusCode::NO_CONTENT)
 }
 
 #[utoipa::path(
@@ -518,8 +498,7 @@ async fn get_group(
 
     let ordering = ordering_params.ordering;
     let settings = pagination_params
-        .validate(1000)?
-        .warn_page_size(100)
+        .validate(100)?
         .into_selection_settings()
         .filter(move || WorkSchedule::WORK_SCHEDULE_GROUP_ID.eq(group_id))
         .order_by(move || ordering.as_work_schedule_ordering());