Skip to content

fix: Konfetti/Abzeichen bei gleicher Sternzahl in Folgerunde + ungenutzte Entwurfsbilder entfernen - #371

Merged
S540d merged 2 commits into
testingfrom
fix/feedback-repeat-rating
Oct 23, 2026
Merged

S540d merged 2 commits into
testingfrom
fix/feedback-repeat-rating

Conversation

@S540d

@S540d S540d commented Oct 23, 2026

Copy link
Copy Markdown
Owner

Pull Request

Beschreibung

Gefunden beim Release-Check testing → main:

  1. Bugfix (services/useGameFeedback.ts): lastCheckedRatingRef wurde nicht zurückgesetzt, wenn das Rating zum Rundenstart auf 0 geht. Folge: Gleiche Sternzahl in zwei aufeinanderfolgenden Runden (z.B. 5 → 5) → in der zweiten Runde kein Konfetti/Sound und keine Abzeichen-Prüfung. Der Fehler existiert bereits in main.
    Neuer Test services/__tests__/useGameFeedback.test.ts reproduziert den Fehler (schlägt ohne Fix fehl) und ist mit Fix grün.
  2. Aufräumen: 9 nirgends referenzierte Entwurfsbilder aus assets/icons/ entfernt (~5 MB; IMG_27xx, ChatGPT Image …, UUID-JPGs). Separater Commit.

Art der Änderung

  • Bugfix (non-breaking change)
  • Neue Funktion (non-breaking change)
  • Breaking Change (Fix oder Feature das bestehende Funktionalität ändert)
  • Dokumentation Update

Cross-Platform Kompatibilität

Betrifft dieser PR plattformspezifischen Code? Nein

Testing Checklist

  • Lokaler Build erfolgreich (expo export Web + Android)
  • Keine TypeScript Errors (npx tsc --noEmit)
  • Keine ESLint Warnings (npm run lint)
  • App startet ohne Crashes (nicht auf Gerät getestet)
  • Geänderte Features funktionieren wie erwartet (Jest: 52 Suites / 482 Tests grün)

Zusätzliche Notizen

Bild-Referenzen geprüft per Volltextsuche über Repo (Code, app.json, Docs, Scripts) — keine Treffer.


Generated by Claude Code

…unde

lastCheckedRatingRef wurde beim Zurücksetzen des Ratings auf 0 (neue Runde)
nicht zurückgesetzt, daher wurde z.B. 5 Sterne nach 5 Sternen übersprungen:
kein Konfetti, keine Abzeichen-Prüfung.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AfCKFu51beRuwRbDYQRaJd
Nirgends referenziert (app.json, Code, Docs, Scripts).

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

Copy link
Copy Markdown

🔀 Mergeability-Report

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

CI-Checks: ✅ keine fehlgeschlagenen Checks

⏳ mergeability / mergeability
⏳ ⚙️ Workflow Linting / actionlint
⏳ 🔧 Platform Compatibility
⏳ 🔐 Docs Privacy Check
⏳ 🔍 Code Quality & Linting
⏳ 🔑 Keystore & Credential Scan
⏳ 🔒 Security Audit
⏳ 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 4c76c49 into testing Oct 23, 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