f047_neu_Edit-der-Namen #28
trachi93
Daniel
jared
Reference in New Issue
Block a user
Delete Branch "f047_neu_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?
Ich habe zum besseren Verständnis ein paar logs noch drin gelassen, aber kannst du danach gerne löschen. Ich hoffe jetzt passt alles soweit...
@@ -1,3 +1,31 @@<script lang="ts">export let href = null;export let type = 'button';type hinzufügen
export let type: 'button' | 'submit' | 'reset' = 'button';
@@ -0,0 +1,94 @@<script lang="ts">Die Componente heißt EditableItem. Das suggeriert, dass damit unterschieldiche Items editierbar sind, also generisch. In meinem Verständins ist das aber eine Componente zugeschnitten für das Editieren des Namens
Habe es umbenannt in NameItemEditor, falls besserer Name bitte eigenständig umbenennen, es wird nur einmal verwendet.
@@ -0,0 +21,4 @@let error: string = $derived(validateName(localName));let manualError = $state('');manualError wird m. E. nicht im code gesetzt. Also es wird nie ein Error definiert, oder?
Habe ich rausgenommen
@@ -0,0 +44,4 @@function commitIfValid() {if (!error && !wasCancelled && localName != currentName) {editedName = localName.trim();Die Funktion commitIfValid setzt editedName = localName.trim();, aber editedName ist ein Prop und sollte nicht direkt überschrieben werden. Stattdessen sollte ein Event ausgelöst werden oder ein Callback genutzt werden.
Ich habe es nun eine neue Variable und dann in onSave übergeben, dabei ist mir die Frage gekommen wie sieht die Abfrage mit Leerzeichen insgesamt aus, habe es als offene Frage ins Backlog geschrieben
@@ -1,3 +1,3 @@import { readFileSync } from 'fs';export default JSON.parse(readFileSync('./config.json').toString());export default JSON.parse(readFileSync('./config_prod.json').toString());Stimmt das wirklich? Ich glaube, dass ist nicht korrekt es sollte config.json bleiben, da der Container auch eine config.json anlegt
@@ -1,7 +1,5 @@<script lang="ts">Wie du beschrieben hast, unnötige Kommentare und console.logs raus
Sollte soweit passen, ansonsten bitte selbstständig rauslöschen
@@ -38,1 +35,3 @@$: err;//Variablen für Modallet open = $state(false);let inProgress = $state(false);In Progress wird nie gesetzt. Daher bisher keine Funktion
Da es zum Modal gehört habe ich es angepasst und nicht gelöscht.
@@ -46,0 +45,4 @@console.log('Eltern, speichern erfolgreich', newName, oldName); //zur besseren Nachvollziehbarkeit noch drin gelassen, kann vorm merge gelöscht werdentry {const res = await fetch(`/api/list/${vorgangToken}/${oldName}`, {//irgendwas stimmt hier nicht, vorgangToken führt zu Fehler in API,Funktioniert es denn jetzt???
Ja, funktioniert.
@@ -0,0 +15,4 @@const crimeNames: string[] = crimesList.map((l) => l.name);if (crimesList.length === 0) {throw redirect(302, '/upload'); // weiterleiten auf die hinzufügen seiteOk, aber warum. Erwarte ich hier nicht dann die Liste??
Ich habe die Umleitung gesetzt, wenn die Liste leer ist, also wenn keine Dateien zum Anzeigen vorhanden sind. So kann man quasi direkt eine Datei neu upload. Wenn nicht gewünscht bitte eigenständig bearbeiten