Skip to content

Fix #211: Copy-&-Paste-Tab entfernen — nur GitHub-Import - #232

Merged
Gree44 merged 1 commit into
stagingfrom
fix/211-remove-manual-upload-tab
Jul 25, 2026
Merged

Fix #211: Copy-&-Paste-Tab entfernen — nur GitHub-Import#232
Gree44 merged 1 commit into
stagingfrom
fix/211-remove-manual-upload-tab

Conversation

@Gree44

@Gree44 Gree44 commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Behebt #211 (Gestaltungsfrage Copy & Paste).

Entscheidung

Es gibt keinen manuellen Upload — Templates werden ausschließlich über die GitHub-Verbindung importiert (das Backend liest die app.yaml direkt aus dem verbundenen Repo). Der zweite Tab „Copy & Paste" war ein dauerhaft deaktivierter Platzhalter (opacity-50 pointer-events-none, „nicht angebunden"-Banner) und hat Dozenten nur verwirrt bzw. die Design-Frage im Issue ausgelöst.

Änderungen (src/components/AddTemplateDialog.tsx)

  • Tab-Leiste entfernt (Tabs/TabsList/TabsTrigger/TabsContent) — der GitHub-Import-Inhalt rendert jetzt direkt im Dialog, ohne Tab-Umschalter.
  • Toter Manual-State entfernt: heatTemplate, cloudInit, appYaml, activeTab sowie der disabledReason-Zweig „Manueller Upload ist nicht angebunden".
  • Ungenutzte Imports entfernt: Tabs*, FileText-Icon.
  • Doku-Kommentare aktualisiert: GitHub ist der einzige Weg; kein Upload.

Unverändert: der komplette GitHub-Flow (Name, Beschreibung, Icon-Upload, URL, app.yaml-Pfad, Sichtbarkeit), Connection-Status-Banner und „GitHub verbinden".

Umfang & Verifikation

  • 1 Datei, +11/−73 (rein subtraktiv, keine Logikänderung am Import).
  • npm run build ✓ (JSX balanciert, keine dangling refs).
  • Backend: keine Änderung nötig — es gab und gibt nur POST /api/v1/templates/import-from-github.

Base: staging.

Templates werden ausschließlich über die GitHub-Verbindung importiert.
Der zweite Tab ("Copy & Paste"/manueller Upload) war ein deaktivierter
Platzhalter ohne Backend-Anbindung und hat nur verwirrt.

- Tab-Leiste (Tabs/TabsList/TabsTrigger) entfernt; der GitHub-Inhalt
  rendert jetzt direkt im Dialog.
- Toten Manual-State (heatTemplate/cloudInit/appYaml, activeTab) und den
  zugehörigen disabledReason-Zweig entfernt.
- Ungenutzte Imports (Tabs*, FileText-Icon) entfernt.
- Doku-Kommentare aktualisiert: GitHub ist der einzige Weg, keinen Upload.
@Gree44
Gree44 merged commit 1affadf into staging Jul 25, 2026
6 checks passed
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.

2 participants