department-for-transport-public / D-TRO

Digital Traffic Regulation Orders (D-TRO)
MIT License
3 stars 1 forks source link

v3.2.3 Schema and examples inconsistency #12

Closed janicki95 closed 1 month ago

janicki95 commented 1 month ago

Lack of consistency between the v3.2.3 schema and examples given.

JSON schema expects the source->section property to be a string[] (array of string values). All examples provided show this property as a string value only. How do you want this property provided?

Example: alt-text-1

Schema: alt-text-2

ruipinge commented 1 month ago

Also detected a few:

stm-john-cooper commented 1 month ago

Thanks for raising this issue. We are reviewing and will update/fix as soon as possible.

ajf2 commented 1 month ago

In JSON-3.2.3-example-point-dtro.json, externalReference should now be a singular object, and its UniqueStreetReferenceNumber property should be an array. I'd argue that all properties should begin with a lowercase letter or at least be consistent, ~but that doesn't really matter.~

It seems the validation on the server is case sensitive and expects to find a Source or Consultation property starting with a capital letter, so consistency would be appreciated!

JHB9876 commented 1 month ago

On behalf of DfT:

Many thanks for raising these issues. The DfT D-TRO team have been reviewing and expect to imminently issue a data specification release (v3.2.4) which aims to address the issues raised.

JHB9876 commented 1 month ago

On behalf of DfT:

Again, many thanks for raising these issues. Many of the issues raised have been addressed in the data specification release 3.2.4. In a small number of cases, the correction requires a code base change to the D-TRO Service and we have recorded the issue and resolution and will address in a future release of the data specification.