Skip to content

Baumbeschreibungen#82

Closed
HelenBielawa wants to merge 2 commits intomainfrom
baumbeschreibungen
Closed

Baumbeschreibungen#82
HelenBielawa wants to merge 2 commits intomainfrom
baumbeschreibungen

Conversation

@HelenBielawa
Copy link
Contributor

No description provided.

@andreascansee
Copy link
Contributor

Bitte Konflikte selbstständig lösen, bevor das gemergt werden kann! ❌

Zusätzlich sollten folgende Punkte vor dem Merge behoben werden:

  • Datenbankzugriff direkt in +page.svelte ist ein Anti-Pattern: Bitte trenne UI und Business-Logik (Separation of Concerns). → Unsere Supabase-Zugriffe sind in frontend/src/lib/supabase gekapselt. Prüfe, ob die benötigte Funktionalität dort bereits existiert – oder lagere sie ggf. dorthin aus.

  • Texte direkt im Code sind schwer wartbar: Bitte lege statische Texte in /static ab und binde sie auf der Seite ein. So können Inhalte später einfacher angepasst werden – auch ohne Codeänderung.

@andreascansee
Copy link
Contributor

Ist wegen #100, der das gewünschte Feature vollständig implementiert, nun obsolet. War aber eine gute Ausgangsbasis. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants