Demo-Modus: Login-Einstieg, Doku, Crash-Fix & Service-Worker-Konflikt - #9
Merged
Merged
Conversation
- demo.ts: 4. Demo-Profil ergaenzt. DEMO_LISTINGS[7] referenzierte OTHER_PROFILES[3], das Array hatte aber nur 3 Eintraege -> beim Import von demo.ts (App-Start) warf OTHER_PROFILES[3]!.id "Cannot read properties of undefined (reading 'id')". - Service Worker: nur noch den von vite-plugin-pwa generierten SW nutzen; Push-Handler aus sw-push.js per workbox.importScripts einbinden und die manuelle Registrierung in main.tsx entfernen (vorher zwei SWs mit Scope /). https://claude.ai/code/session_01FVY3E3QvCEMjXz9qMLgyj2
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Überblick
Dokumentiert den Demo-Modus, verbessert den Demo-Einstieg auf der Login-Seite und behebt zwei Fehler, die auf dem Vercel-Deployment auftraten.
Änderungen
Login-Integration verbessern (
src/components/auth/login-form.tsx)Button(variant="outline", volle Breite): „Ohne Login ausprobieren →" mit Hinweistext. Logik inhandleDemoausgelagert.Projektplan dokumentieren (
docs/Kommit_Projektplan.md)Crash-Fix
Cannot read properties of undefined (reading 'id')(src/lib/demo.ts)OTHER_PROFILES[3], das Array hatte aber nur 3 Einträge →OTHER_PROFILES[3]!.idwarf beim Import vondemo.ts(App-Start überuse-auth.ts→AuthProvider) den TypeError. Fehlenden 4. Demo-User ergänzt, sodass alle verwendeten Indizes (0–3) existieren.Service-Worker-Konflikt auflösen (
vite.config.ts,src/main.tsx)/(manuell registriertessw-push.js+ der vonvite-plugin-pwagenerierte). Jetzt nur noch ein SW (/sw.js), der die Push-Handler viaworkbox.importScripts(["sw-push.js"])einbindet; manuelle Registrierung inmain.tsxentfernt.Verifikation
pnpm type-checksauber,pnpm lint0 Fehler (nur vorbestehende Warnings),pnpm builderfolgreich.registerSW.jsregistriert nur/sw.js, undsw.jsenthältimportScripts("sw-push.js").Nicht enthalten
Der
manifest.webmanifest401 ist Vercel Deployment Protection auf der Preview-URL (kein App-Bug) — abschaltbar unter Settings → Deployment Protection bzw. auf der Produktiv-Domain irrelevant.https://claude.ai/code/session_01FVY3E3QvCEMjXz9qMLgyj2
Generated by Claude Code