Add Validations to JSON Schema Properties #36
Conversation
maikwoehl-psi
added
wait-for-approval
|
Hallo @DohmenClaus, wie lautet hierzu die Abstimmung? Angesichts der letzten Diskussionen bin ich mir nicht sicher, ob die Einführung einer Restriktion einen "Breaking Change" darstellt. Ich bin gewillt zu sagen, dass dadurch zwar die Implementierung ggbf. angepasst werden muss, diese dafür dann aber stabiler ist. Für den Fall, dass irgendjemand Gleitkommazahlen statt Ganzzahlen von 0 bis 100 verwendet hat, ist das bestimmt eine sinnvolle Verbesserung. Viele Grüße Maik Wöhl |
|
Vielen Dank für deinen Kommentar und die Anmerkungen, @maikwoehl-psi. Die Festlegung eines Wertebereichs durch Minimum und Maximum dient lediglich zur Begrenzung der Werte, ändert jedoch nicht den zugrunde liegenden Datentyp. Wenn ich nachfolgendes aber richtig verstehe:
Impliziert deine Nachricht, dass der zugrunde liegende Datentyp selbst von |
maikwoehl-psi
added
accepted
discussed
|
@Basler182 Eine Anpassung des Datentyps sehen wir für die Version 1.1.0 aktuell nicht vor. |
* Add v1.1 JSON schemas fixing issues #2 and #3 * Fixes #5 * removing "uri" format constraint from all string identifiers as agreed in the VDV 463 meeting * removing "uri" format constraint from all string identifiers * removing "uri" format constraint from all string identifiers * Remove obsolete whitespace changes * #20: Allow empty arrays * #22: Allow empty ChargingPredictionData * #22: Add structure ChargingPredictionDataDepartureTime to ChargingPredictionData * #22: Fix descriptions * Update v1.1/ProvideChargingInformationRequest.json Co-authored-by: Maik Wöhl <[email protected]> * Update v1.1/ProvideChargingInformationRequest.json Co-authored-by: Maik Wöhl <[email protected]> * Add critical condition for tractionBattery, add hydrogentankinfo (#16) * Add critical condition for tractionBattery, add hydrogentankinfo * Move vehicleInfo into it's own list * Add VehicleId to ChargingPointInfo to reference it * Switch criticalCondition to integer to reflect SNA values * Revert "Add VehicleId to ChargingPointInfo to reference it" This reverts commit d14814e. Revert "Move vehicleInfo into it's own list" This reverts commit 0aa89ce. * Rename criticalCondition to status * Revert "Add v1.1 JSON schemas fixing issues #2 and #3" * Add evccId to VehicleInfo structure Closes #30 * Revert "Add critical condition for tractionBattery, add hydrogentankinfo (#16)" This reverts commit c82bf61. * Add Validations to JSON Schema Properties (#36) * Move changes of v1.1 to v1.0 Signed-off-by: Maik Wöhl <[email protected]> * Rename v1.0 to schema Signed-off-by: Maik Wöhl <[email protected]> --------- Signed-off-by: Maik Wöhl <[email protected]> Co-authored-by: Kowatsch, Benjamin <[email protected]> Co-authored-by: Dominique Müller <[email protected]> Co-authored-by: Benjamin Kowatsch <[email protected]> Co-authored-by: DohmenClaus <[email protected]> Co-authored-by: Kilian Schneider <[email protected]>
The current JSON schemas lack validations for numerical properties. These validations ensure that the data conforms to expected value ranges. This aims to enhance the robustness and accuracy of the schema definitions.