Skip to content

Commit

Permalink
feat: deletion of day plan todos is now possible
Browse files Browse the repository at this point in the history
  • Loading branch information
Carsten Koch committed Jun 4, 2024
1 parent b24e58d commit a9e6831
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 21 deletions.
6 changes: 5 additions & 1 deletion api/useDayplans.ts
Original file line number Diff line number Diff line change
Expand Up @@ -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,
Expand Down
6 changes: 0 additions & 6 deletions components/dayplan/dayplan-form.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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);
};

Expand Down
3 changes: 0 additions & 3 deletions components/dayplan/task-form.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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();
};
Expand Down
12 changes: 1 addition & 11 deletions docs/releases/next.md
Original file line number Diff line number Diff line change
Expand Up @@ -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)

0 comments on commit a9e6831

Please sign in to comment.