Fix date test #292

[TobiasNx]

Wenn man bash test.sh ausführt, knallen die Test im Gegensatz zu npm run tests durch die Anpassung im Test, validiert auch das dateTime Beispiel bei bash test.sh

[acka47]

Diesen PR und #292 können wir dann schließen, oder?

[acka47]

Hmm, vielleicht doch nicht. Mit neueren ajv-formats-Versionen würde es dann Probleme geben, weil ab Version 3 bei der Nutzung von "Format": "date-time"` die Zeitzone – entgegen der ISO-Spezifikation – nicht mehr optional ist, wenn ich ajv-validator/ajv-formats#54 (comment) richtig verstehe.

[TobiasNx]

Wir könnten auch das format: iso-date-time nehmen: https://github.com/ajv-validator/ajv-formats/releases

[acka47]

Wenn wir eine neuere ajv-formats-version nutzen und die Beispieldateien anpassen, müssten wir auch dateCreated und dateModified in der Spec anpassen. Das wäre auch ein Breaking Change. Von daher würde ich das eher lassen.

[acka47]

Wir könnten auch das format: iso-date-time nehmen: https://github.com/ajv-validator/ajv-formats/releases

Ah, cool. Das wäre dann die beste Lösung, denke ich.

[acka47]

Ok, dann lass uns doch den PR hier und das dazugehörige Ticket schließen und die Dinge bei Gelegenheit in #296 abhandeln, oder?

[TobiasNx]

Ok, dann lass uns doch den PR hier und das dazugehörige Ticket schließen und die Dinge bei Gelegenheit in #296 abhandeln, oder?

Meinetwegen können wir das gerne so machen. Ich hatte ja auch noch einen Draft, der hier aufseitzt und die JSON-SCHEMA Version hochzieht, da wir keine breaking "components" haben, den würde ich dann noch mal unabhängig davon aufsetzen.

[TobiasNx]

Ich schließe den PR, da das Ticket im Rahmen von #296 gelöst wird.