Skip to content

Commit 641b12f

Browse files
committed
Updated todo list
1 parent 7906ab0 commit 641b12f

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

EditorsDraft/examples.js

+3-2
Original file line numberDiff line numberDiff line change
@@ -49,10 +49,9 @@ function dataExampleOrderQuoteCreationErrorResponse(utils, content) {
4949

5050
//TODO: Should we have a separate type for OrderRequest to better define the required params?
5151

52-
//TODO: Include identifier in OrderItem and include logic to specify that it is reflected back.
5352
//TODO: Force OrderQuote to reflect back everything, so that it can be passed back and forth, same for Order, noting that it doesn't actually need to be stored anywhere.
5453

55-
//TODO: Add something about price checking?
54+
//TODO: Add something about price checking? [DONE]
5655
//TODO: - You're supposed to check price is the same, so the completed OrderQuote must be passed into the Order in order to complete the Order - does this get around any reflection issues?
5756
// All supported properties are reflected (and stored if Order GET is implemented), however the feed is a PATCH, so essential properties should be stored by the broker and periferal properties don't need to be stored by the Booking System for more accessible implementation.
5857

@@ -99,6 +98,8 @@ function dataExampleOrderQuoteCreationErrorResponse(utils, content) {
9998

10099
// Make "opportunity" a key word in the spec
101100

101+
// How do we know which status code to use when multipe errors have conflicting status codes? Do we have an example with just one error and no array?
102+
102103
function dataExampleOrderCreationRequest(utils, content) {
103104
return generateRequest("PUT", API_PATH + "/orders/" + UUID, OPERATIONS_MEDIA_TYPE, {
104105
"@context": CONTEXT,

0 commit comments

Comments
 (0)