[cleanup, refactoring] DTO Umbenennung value -> zone

[clean-coder]

In den beiden DTOs (im Server):

• KeyResultLastCheckInOrdinalDto.java
• CheckInOrdinalDto.java

gibt es jeweils ein "Property" Zone value, was nicht so schön ist. Besser wäre Zone zone. Die Umbenennung im Server führt dazu, dass die Cypress Tests rot werden, da es im Client auch entsprechende Modell Klassen gibt, die auch Zone value verwenden.

• CheckIn.ts
• und ev. noch mehr

Zone zone wäre schöner, aber man muss das im Client und im Server gleichzeitig refactoren.

Anforderungen

• Die Properties im Backend heissen neu zone, um sie sprechender zu machen
• Das Frontend wird so angepasst, wie nötig

Aktzeptanzkriterien

• Das Property in KeyResultLastCheckInOrdinalDto.java wurde umbenannt
• Das Property in CheckInOrdinalDto.java wurde umbenannt
• Die Properties im Frontend wurden umbenannt (z.B. in CheckInMin.ts und CheckIn.ts
• Unit und E2E-Tests wurden angepasst falls nötig

[clean-coder]

In den beiden DTOs (im Server):

• KeyResultLastCheckInOrdinalDto.java
• CheckInOrdinalDto.java

gibt es jeweils ein "Property" Zone value, was nicht so schön ist. Besser wäre Zone zone. Die Umbenennung im Server führt dazu, dass die Cypress Tests rot werden, da es im Client auch entsprechende Modell Klassen gibt, die auch Zone value verwenden.

• CheckIn.ts
• und ev. noch mehr

Zone zone wäre schöner, aber man muss das im Client und im Server gleichzeitig refactoren.

[Miguel7373]

Todo

• Rename value to zone in Backend
• Create new models for metric and ordinal CheckIn's
• Change logic so it works like the keyResult's

[Miguel7373]

Stand 26.11.2024

Das Grundprinzip funktioniert. Ich habe es nun mit mehreren Models im Frontend gelöst und dann die Logik so angepasst, dass es überall wieder richtig funktioniert und unterscheidet, ob es metric oder ordinal ist (nur wo es nötig was)

Still ToDo

• Tests auf der Pipe zum Laufen bringen
• Im Hintergrund wird der Streifen, ob es im Fail Commit oder Target Bereich ist, noch nicht angezeigt (nur bei ordinalen)
• Generelles code clean up, da es noch ein bisschen Buggy scheint.

[Miguel7373]

Stand 26.11.2024

Ich hatte noch ein Problem mit dem Scoring im Hintergrund und habe das mit einem pars mal gefixt was nicht so schön ist.
Es ust um keyresult componet drin damit es nur für den hintergrund würkt da von dah das key result her kommt

Still ToDo

• In dem KeyResult sollte das atribut lastCheckIn warscheindlich in die unterklassen geschoben werden
• Parsing muss generell überarebitet werden
• Für das parsing Scoring im Hintergrund soll eine bessere lösung gefunden werden
• Tests laufen noch nicht durch die müssen auch noch angepasst werden

[Miguel7373]

Stand 3.12.2024

Das Ticket ist nun fast fertig, die Funktionen laufen alle wieder.

Still ToDo:

• Rebase
• Das Type-Assertion mit as im Frontend ist durch das Aufsplitten des Attributs lastCheckIn in den keyresults an ein paar Stellen unnötig geworden, überprüfe dies und schau auch noch, ob das Type-Assertion nicht auch effizienter gelöst werden kann.
• Die Edit-Tests im key-result.cy.ts File failen alle noch
• Frontend-Unit-Tests laufen noch nicht, die müssen noch an die neue Logik angepasst werden