Ich habe den Pull Request zum neuen Filtermodul geprüft. Die Grundidee ist gut und die Komponenten sind sauber getrennt. Ein Punkt macht mir aber noch Sorgen: Der aktuelle State wird an mehreren Stellen direkt verändert. Dadurch kann später schwer nachvollzogen werden, warum sich einzelne Filterwerte unerwartet ändern. Besser wäre es, die Updates in eine eigene Funktion zu ziehen und dort alle Änderungen zentral zu behandeln.
Glossar
- der Pull Request
- Anfrage zur Code-Überprüfung und Zusammenführung.„Ich habe den Pull Request zum neuen Filtermodul geprüft.“
- die Oberfläche
- sichtbarer Teil der App„Im Moment bleibt die Oberfläche einfach leer, wenn der Request fehlschlägt.“
- der Nutzer
- Person, die die App benutzt„Für Nutzer wirkt das wie ein stiller Fehler.“
- sauber
- klar und ordentlich umgesetzt„Die Grundidee ist gut und die Komponenten sind sauber getrennt.“
- der Fallback
- Ersatzlösung bei Fehler„Bitte ergänze dafür eine sichtbare Meldung und einen kleinen Fallback.“