From a9e6831435e034f04649ab843b1a1289a7be9407 Mon Sep 17 00:00:00 2001
From: Carsten Koch <carstenbkoch@gmail.com>
Date: Wed, 5 Jun 2024 01:13:26 +0200
Subject: [PATCH] feat: deletion of day plan todos is now possible

---
 api/useDayplans.ts                  |  6 +++++-
 components/dayplan/dayplan-form.tsx |  6 ------
 components/dayplan/task-form.tsx    |  3 ---
 docs/releases/next.md               | 12 +-----------
 4 files changed, 6 insertions(+), 21 deletions(-)

diff --git a/api/useDayplans.ts b/api/useDayplans.ts
index 9a3ce8439..6feb4e56f 100644
--- a/api/useDayplans.ts
+++ b/api/useDayplans.ts
@@ -94,7 +94,11 @@ const mapDayPlan: (dayplan: DayPlanData) => DayPlan = ({
       doneOn: doneOn ? new Date(doneOn) : undefined,
       projectId: project?.id,
     }))
-    .sort((a, b) => a.createdAt.getTime() - b.createdAt.getTime()),
+    .sort((a, b) =>
+      a.done === b.done
+        ? a.createdAt.getTime() - b.createdAt.getTime()
+        : (a.done ? 1 : 0) - (b.done ? 1 : 0)
+    ),
   projectTasks: projectTasks
     .map(({ id, task, done, createdAt, projects }) => ({
       id,
diff --git a/components/dayplan/dayplan-form.tsx b/components/dayplan/dayplan-form.tsx
index ea5a9a32e..1cc22c9c4 100644
--- a/components/dayplan/dayplan-form.tsx
+++ b/components/dayplan/dayplan-form.tsx
@@ -18,7 +18,6 @@ import {
 } from "../ui/form";
 import { Input } from "../ui/input";
 import { Popover, PopoverContent, PopoverTrigger } from "../ui/popover";
-import { useToast } from "../ui/use-toast";
 
 const FormSchema = z.object({
   goal: z
@@ -49,13 +48,8 @@ const DayPlanForm: FC<DayPlanFormProps> = ({
       date: getHours(new Date()) < 12 ? new Date() : addDays(new Date(), 1),
     },
   });
-  const { toast } = useToast();
 
   const handleSubmit = ({ date, goal }: z.infer<typeof FormSchema>) => {
-    toast({
-      title: "You created a new day plan.",
-      description: `Your goal for ${date.toLocaleDateString()} is "${goal}".`,
-    });
     onSubmit(goal, date);
   };
 
diff --git a/components/dayplan/task-form.tsx b/components/dayplan/task-form.tsx
index b9f0931ca..6c88f5240 100644
--- a/components/dayplan/task-form.tsx
+++ b/components/dayplan/task-form.tsx
@@ -13,7 +13,6 @@ import {
   FormMessage,
 } from "../ui/form";
 import { Input } from "../ui/input";
-import { useToast } from "../ui/use-toast";
 
 const FormSchema = z.object({
   todo: z
@@ -36,10 +35,8 @@ const TaskForm: FC<TaskFormProps> = ({ createTodo }) => {
     resolver: zodResolver(FormSchema),
     defaultValues: { todo: "", projectId: "" },
   });
-  const { toast } = useToast();
 
   const handleSubmit = ({ todo, projectId }: z.infer<typeof FormSchema>) => {
-    toast({ title: "You created a new task", description: "new task created" });
     createTodo(todo, projectId !== "" ? projectId : undefined);
     form.reset();
   };
diff --git a/docs/releases/next.md b/docs/releases/next.md
index 063a4a414..e143b49c4 100644
--- a/docs/releases/next.md
+++ b/docs/releases/next.md
@@ -4,15 +4,5 @@
 - Aufgabenliste umstellen auf Checkbox
 - Editieren von Einträgen in Aufgabenliste ermöglichen
 - Löschen von Einträgen ermöglichen (in Today's Tasks; immer zur Sicherheit abfragen)
-
-In Arbeit:
-
-- Löschen von Einträgen ermöglichen (Meetings und bei Notizen; immer zur Sicherheit abfragen)
-
-- Hinweise bei den Tagesplänen, wenn offene Pläne in anderen Kontexten existieren
-
-- Nochmal prüfen, was bei der Migration passiert, ob ich dem Alert traue; dann kann ich migrieren und entsprechend die Logik für die Legacy Tasks endlich entfernen
-
+- Aufgaben im Tagesplan sortieren (erledigte nach unten, dann nach Erstelldatum)
 - Die Bestätigung, dass eine Aufgabe im Tagesplan angelegt wurde, ist unnötig
-
-- Aufgaben im Tagesplan sortieren (erledigte nach unten, ansonsten nach Projekt Prio sortieren, dann nach Erstelldatum)