Skip to content

fix: Galerie-Status bei Neustart ab Level 1, Ergebnis-Feedback pro Runde stabil - #357

Merged
S540d merged 1 commit into
testingfrom
fix/restart-and-feedback
Oct 12, 2026
Merged

S540d merged 1 commit into
testingfrom
fix/restart-and-feedback

Conversation

@S540d

@S540d S540d commented Oct 11, 2026

Copy link
Copy Markdown
Owner

Pull Request

Beschreibung

Zwei Funde aus dem Simplify-Audit (#356), vom Nutzer zur Korrektur freigegeben:

  1. restartFromLevel1 hat als einziger Reset-Pfad savedToGallery nicht zurückgesetzt. Nach „Von Level 1 neu starten“ konnte „In Galerie speichern“ deshalb gesperrt bleiben. Jetzt verhält er sich wie die anderen Reset-Pfade.
  2. Das Ergebnis-Feedback wechselte bei jedem Re-Render. ResultPhase hat getRatingFeedback() im Render aufgerufen, und die Funktion rotiert durch Nachrichten. Jetzt wird das Feedback per useMemo einmal pro Bewertung gewählt. Außerdem werden die Texte für 1–2 Sterne als DE/EN-Paar mit einem gemeinsamen Index geholt. Vorher gab es zwei Aufrufe, der Zähler lief doppelt und die beiden Sprachen liefen auseinander.

Art der Änderung

  • Bugfix (non-breaking change)
  • Neue Funktion
  • Breaking Change
  • Dokumentation Update

Cross-Platform Kompatibilität

Betrifft dieser PR plattformspezifischen Code? Nein

Testing Checklist

  • Keine TypeScript Errors, keine ESLint Warnings, Prettier sauber
  • npm test: 460/460 grün
  • 2 neue Regressionstests (Galerie-Reset, Feedback-Auswahl nur einmal pro Bewertung). Beide schlagen gegen den alten Code fehl.

🤖 Generated with Claude Code

https://claude.ai/code/session_017fxbsNALrmwMj7LGMDPLw2


Generated by Claude Code

…edback pro Runde festhalten

- useGamePhase.restartFromLevel1 setzt savedToGallery zurück (wie die
  anderen Reset-Pfade)
- ResultPhase wählt das Feedback per useMemo einmal pro userRating statt
  bei jedem Render (getRatingFeedback rotiert durch Nachrichten)
- RatingManager: 1-2-Sterne-Texte als DE/EN-Paar mit einem Index
  (vorher zwei Aufrufe, Zähler lief doppelt, Sprachen liefen auseinander)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017fxbsNALrmwMj7LGMDPLw2
@github-actions

Copy link
Copy Markdown

🔀 Mergeability-Report

✅ Konfliktfrei mergebar (unstable).
✅ Ziel-Branch testing ist policy-konform.

CI-Checks: ✅ keine fehlgeschlagenen Checks

⏳ ⚙️ Workflow Linting / actionlint
⏳ 🔧 Platform Compatibility
⏳ 🔑 Keystore & Credential Scan
⏳ 🔍 Code Quality & Linting
⏳ 🔒 Security Audit
⏳ mergeability / mergeability
⏳ 🔐 Docs Privacy Check
⏳ Analyze (javascript-typescript)
⏳ Analyze (actions)

Merge-Checkliste

  • CI-Checks grün
  • Keine Merge-Konflikte
  • Ziel-Branch korrekt
  • (Optional) tiefes KI-Review via /review aus Claude Code (Pro-Abo, ohne API-Kosten)

Kostenlos generiert von reusable-mergeability.yml · keine API-Kosten · review-gate: success

@S540d
S540d merged commit 8457f99 into testing Oct 12, 2026
14 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