Metric Check-In "Neuer Wert" wird nicht korrekt validiert

[RandomTannenbaum]

Das Feld "Neuer Wert" beim Erstellen eines Check-Ins wird nicht korrekt validiert. Eigentlich sollten nur numerische Werte zugelassen sein, doch das Feld lässt auch Text zu. Sendet man das Form dann ab, so werden einfach alle nicht-numerischen Zeichen entfernt. Der Wert setzt sich dann ausden übriggebliebenen Zahlen zusammen oder falls keine Zahl vorhanden war, ist er einfach 0.

Beispiele:

• "Test" wird zu 0
• "Test123" wird zu 123
• "T1e3st4" wird zu 123

Anforderungen

• Das Feld "Neuer Wert" beim Erstellen eines neuen Check-Ins lässt nur numerische Werte zu.
• Eine entsprechende Fehlermeldung informiert den Benutzer beim Eingeben nicht-numerischer Werte.
• Whitespaces links oder rechts eines nummerischen Werts sind valid und werden automatisch entfernt.
• Whitespaces zwischen Zahlen sind nicht valid

Akzeptanzkriterien

• Alle Anforderungen sind erfüllt
• Das Feld wird korrekt validiert und lässt nur Werte zu, die numerisch sind
• Whitespaces werden automatisch entfernt, wenn sie ganz links oder rechts sind
• Tests sind ergänzt

[RandomTannenbaum]

ToDo

• Anschauen warum das Feld nicht korrekt validiert wird
• Die Validators auf der Formgroup so anpassen, dass Zahlen enforced werden und eine entsprechende Fehlermeldung angezeigt wird
• Tests schreiben die dies überprüfen
• Peer-review
• Review

[RandomTannenbaum]

Stand 22.01.2025
Ich habe das metric-check-in-directive in Absprache mit Pipo entfernt. Dieses hat zum oben beschriebenen Verhalten geführt, bei dem Zahlen, die in einem String vorhanden sind, aus diesem extrahiert werden. Nun sind nur noch Zahlen zugelassen. Ausserdem gibt es keinen Fehler mehr, wenn man vor oder hinter eine Zahl Leerzeichen schreibt.

Um das Verhalten zu Testen habe ich Unit- und E2E-Tests ergänzt.

Was noch fehlt
Pipo hat noch gefragt ob bei "Letzter Wert" zwischen der Zahl und der Währung noch ein Leerzeichen kommt und ob es richtig platziert ist. Das sollte noch kurz im Figma angeschaut werden.

[peggimann]

Der Abstand zwischen Zahl und Einheit ist im Figma nicht definiert, wir fragen das nach bei UX und nehmen es in die Story #1320