BUG OrderItem['@id'] should not be marked as REQUIRED for Broker Requests #235
Description
In the most recent version of the spec, OrderItem['@id'] is marked as REQUIRED for Broker Requests (https://openactive.io/open-booking-api/EditorsDraft/1.0CR3/#schema-orderitem).
This is not correct. The Booking System generates its own OrderItem IDs at B (and also at P assuming this issue is agreed upon).
So it should look like:
| Property | Broker Request | Booking System Response | Type | Notes |
|---|---|---|---|---|
@id |
- | REQUIRED | https://schema.org/URL |
A URI providing a unique identifier ... |
Additional Suggestion: Booking System Response: REQUIRED*
| Property | Broker Request | Booking System Response | Type | Notes |
|---|---|---|---|---|
@id |
- | REQUIRED* | https://schema.org/URL |
A URI providing a unique identifier ... *This property is required for B responses and ... |
I have a feeling a lot of implementers skim through this table. It would be good to signpost things as much as possible. By including the asterisk, it may make it clearer to them that it is not required in all cases. This may be a bigger issue though as there are many properties which may warrant a similar treatment