innohub/tatort
7
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

f047_Edit-der-Namen #15

Closed
jared wants to merge 11 commits from f047_Edit-der-Namen into development
pull from: f047_Edit-der-Namen
merge into: innohub:development
Conversation 1 Commits 11 Files Changed 18 +598 -210
jared commented 2025-06-30 07:05:55 +02:00
Owner
Copy Link
No description provided.
jared added 2 commits 2025-06-30 07:05:55 +02:00
Frontend passt wieder f8234c0488
erstellt EditableItem mit frontend-error-handling eingefügt caselist, crimelist f87b106ad2
jared reviewed 2025-06-30 08:08:46 +02:00
src/lib/components/EditableItem.svelte Outdated
@@ -0,0 +11,4 @@
}
export let value: string = '';
export let variant: '' | 'casename' | 'crimename' = ''; // casename | crimename
jared commented 2025-06-30 07:58:31 +02:00
Author
Owner
Copy Link

Der Kommentar könnte m.M.n. raus

Der Kommentar könnte m.M.n. raus
mina marked this conversation as resolved
src/lib/components/EditableItem.svelte Outdated
@@ -0,0 +17,4 @@
export let editable: boolean = true;
export let editing: boolean;
console.log('Debug editing', editing);
jared commented 2025-06-30 07:08:51 +02:00
Author
Owner
Copy Link

Bitte Console.log entfernen.

Bitte Console.log entfernen.
mina marked this conversation as resolved
src/lib/components/EditableItem.svelte Outdated
@@ -0,0 +22,4 @@
const existingNames = existings.map((item) => item.name);
const dispatch = createEventDispatcher<{
editSart: {};
jared commented 2025-06-30 07:10:56 +02:00
Author
Owner
Copy Link

Typo editSart to editStart

Typo editSart to editS**t**art
mina marked this conversation as resolved
src/lib/components/EditableItem.svelte Outdated
@@ -0,0 +27,4 @@
delete: {};
cancel: void;
}>();
jared commented 2025-06-30 07:15:16 +02:00
Author
Owner
Copy Link

The signature '(): EventDispatcher<{ editSart: {}; save: {}; delete: {}; cancel: void; }>' of 'createEventDispatcher' is deprecated.

Mittlerweile wird ein Event mittels Callback function dispatched. Gucke ggf. noch einmal in die Docs.

Zum Beispiel:

<script type="ts"> const props: { onClick(): void; onExplode(name: string, age: number): void; } = $props(); </script>

<MyComponent
onClick={() => alert('clicked')}
onExplode={(name, age) => alert(name + ' ' + age)}
/>

The signature '(): EventDispatcher<{ editSart: {}; save: {}; delete: {}; cancel: void; }>' of 'createEventDispatcher' is deprecated. Mittlerweile wird ein Event mittels Callback function dispatched. Gucke ggf. noch einmal in die Docs. Zum Beispiel: <script type="ts"> const props: { onClick(): void; onExplode(name: string, age: number): void; } = $props(); </script> <MyComponent onClick={() => alert('clicked')} onExplode={(name, age) => alert(name + ' ' + age)} />
mina commented 2025-06-30 16:38:51 +02:00
Owner
Copy Link

Ich habe es angepasst, es musste einiges verändert werden, bitte schaue es dir nochmal an.

Ich habe es angepasst, es musste einiges verändert werden, bitte schaue es dir nochmal an.
jared marked this conversation as resolved
src/lib/components/EditableItem.svelte Outdated
@@ -0,0 +28,4 @@
cancel: void;
}>();
let internalValue = value;
jared commented 2025-06-30 07:23:25 +02:00
Author
Owner
Copy Link

Villt die Variable noch besser bennennen. InternalValue oder oldValue, kann halt alles sein...

Villt die Variable noch besser bennennen. InternalValue oder oldValue, kann halt alles sein...
mina commented 2025-06-30 16:42:22 +02:00
Owner
Copy Link

Allgemein, weil es eine Komponente ist, die vielseitig einsetzbar. Es geht darum quasi den Startwert? zu speichern, für den Abbruch und den aktuellen Wert, um ein editieren in Gang zu bringen. Man könnte es auch cancelValue und EditValue nennen? Vielleicht besser?

Allgemein, weil es eine Komponente ist, die vielseitig einsetzbar. Es geht darum quasi den Startwert? zu speichern, für den Abbruch und den aktuellen Wert, um ein editieren in Gang zu bringen. Man könnte es auch cancelValue und EditValue nennen? Vielleicht besser?
jared commented 2025-07-21 14:04:20 +02:00
Author
Owner
Copy Link

Naja, der Begriff Value bringt halt nicht, weil das quasi immer ein value ist. Daher so explizit bennen wie es geht

Naja, der Begriff Value bringt halt nicht, weil das quasi immer ein value ist. Daher so explizit bennen wie es geht
jared marked this conversation as resolved
src/lib/components/EditableItem.svelte Outdated
@@ -0,0 +34,4 @@
let duplicate = false;
let errors: string[] = [];
let errorText = '';
jared commented 2025-06-30 07:21:25 +02:00
Author
Owner
Copy Link

Wird im weiteren Verlauf nicht verwendet, daher löschen

Wird im weiteren Verlauf nicht verwendet, daher löschen
mina marked this conversation as resolved
src/lib/helper/error-utils.ts
@@ -0,0 +6,4 @@
errors.push('Name existiert bereits');
return errors;
}
jared commented 2025-06-30 08:04:31 +02:00
Author
Owner
Copy Link

Grundsätzlich wird hier der Value in diesem Fall ein Name validiert. Es ist keine generische validateInput Funktion. Daher sollte sie entsprechend benannt werden. Darüber hinaus wird sie nur in EditableItem verwendet. Daher ist ggf. ein Auslagern nicht sinnvoll, oder soll sie noch wo anders verwendet werden?

Grundsätzlich wird hier der Value in diesem Fall ein Name validiert. Es ist keine generische validateInput Funktion. Daher sollte sie entsprechend benannt werden. Darüber hinaus wird sie nur in EditableItem verwendet. Daher ist ggf. ein Auslagern nicht sinnvoll, oder soll sie noch wo anders verwendet werden?
mina commented 2025-06-30 17:44:23 +02:00
Owner
Copy Link

Werde ich nochmal überarbeiten

Werde ich nochmal überarbeiten
src/routes/(angemeldet)/list/+page.svelte
@@ -7,2 +6,3 @@
export let editingId: number;
const caseList = data.caseList;
interface ListItem {
jared commented 2025-06-30 08:05:27 +02:00
Author
Owner
Copy Link

Man könnte einen Ordner für type anlegen src/lib/types/...

Man könnte einen Ordner für type anlegen src/lib/types/...
mina commented 2025-06-30 17:43:50 +02:00
Owner
Copy Link

s.u.

s.u.
src/routes/(token-based)/list/[vorgang]/+page.svelte Outdated
@@ -18,4 +14,3 @@
/** @type {import('./$types').PageData} */
export let data;
interface ListItem {
jared commented 2025-06-30 08:07:02 +02:00
Author
Owner
Copy Link

s. o.

s. o.
mina commented 2025-06-30 17:40:23 +02:00
Owner
Copy Link

Ja, das können wir mal im Daily ansprechen, ich habe ein Ticket daraus gemacht.

Ja, das können wir mal im Daily ansprechen, ich habe ein Ticket daraus gemacht.
mina added 1 commit 2025-06-30 17:48:15 +02:00
tagesupdate: dispatch überarbeitet, $props überarbeitet 1c471b4239
mina added 1 commit 2025-07-08 09:32:22 +02:00
hochladen f335f7a13f
mina added 1 commit 2025-07-08 14:48:09 +02:00
zwischenstand: bug delete tatort f5136c2924
mina added 2 commits 2025-07-08 16:20:20 +02:00
Merge branch 'development' into f047_Edit-der-Namen, bug gefixt 2fbe2bf77e
bug fixed 7ad52618fb
mina added 1 commit 2025-07-10 10:25:07 +02:00
überarbeitung edit tatort funktioniert, edit vorgang bis api auch, aber dann hakt es
Some checks failed
InnoHub Processor/tatort/pipeline/head This commit was not built
Details
InnoHub Processor/tatort/pipeline/pr-development There was a failure building this commit
Details
0b4d189439
jared added 1 commit 2025-07-25 08:24:03 +02:00
deleted some comments and console.log 631ce77f7d
mina added 1 commit 2025-08-06 15:26:27 +02:00
testing merge conflict 4d50cd369b
mina added 1 commit 2025-08-07 23:09:35 +02:00
Kommentare hinzugefügt 66482f0feb
mina closed this pull request 2025-08-12 17:37:53 +02:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
jared
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
christoph Daniel git jared mina titus trachi93
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: innohub/tatort#15
Delete Branch "f047_Edit-der-Namen"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?