Skip to content

Conversation

@tkarstaedt
Copy link
Collaborator

@tkarstaedt tkarstaedt commented Nov 10, 2025

Fix: ChargingProgressPrediction Typos and Improvements

Fixes #29 (supersedes #55)

Changes

  • Fixed typo: ChargingProgressPredicitionChargingProgressPrediction
  • Fixed typo: ChargingProgressPredicitionListEntryChargingProgressPredictionListEntry
  • Added missing descriptions for ChargingProgressPredictionListEntry object
  • Added descriptions for predictedSoc and predictedTime properties
  • Added min/max constraints for predictedSoc (0-100)
  • Added additionalProperties: false at object level for better schema validation
  • Improved all descriptions for better clarity

Related

Validation

  • JSON Schema is valid
  • All property descriptions are clear and complete

Torsten Karstaedt added 2 commits November 11, 2025 00:16
…DVde#29, supersedes VDVde#55)

- Fix typo: 'Predicition' -> 'Prediction' in names and references
- Add missing description for ChargingProgressPredictionListEntry
- Add descriptions for predictedSoc and predictedTime properties
- Add min/max constraints for predictedSoc (0-100)
- Add additionalProperties: false at object level
- Improve descriptions for better clarity
@BenjaminKowatschVector BenjaminKowatschVector added the specification Improvements or additions to specification label Jan 27, 2026
Copy link
Collaborator

@BenjaminKowatschVector BenjaminKowatschVector left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hallo Torsten,
wie besprochen sollten wir noch die Anzahl an Elementen begrenzen.
Ich habe bereits ein Änderungsvorschlag dafür erstellt.

"items": {
"$ref": "#/definitions/ChargingProgressPredictionListEntry"
},
"additionalProperties": false
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"additionalProperties": false
"minItems": 0,
"maxItems": 10,
"additionalProperties": false

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

specification Improvements or additions to specification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Prognose Ladeprozeß

2 participants