Closed larsolofsson closed 5 months ago
Decisions on 2023-12-20
Done in commit d12ef5a.
Done in commit d12ef5a.
Done in commit d12ef5a.
OK, that means that salesOrderNumber
should be removed from ListOfSupplierOrders
which contains only "header" information. The removal is done in commit d12ef5a. The move is done in commit bcda087: The salesOrderNumber
is not required within supplierOrderLineItems
, is it OK?
What about the purchaseOrderNumber
? Should it also be moved to the supplierOrderLineItems
(in GetSupplierOrderById
)?
In addition to that, we might want to also consider the following properties:
salesOrderTimestamp
salesOrderStatus
salesOrderLineItemNumber
salesOrderLineItemStatus
purchaseOrderTimestamp
purchaseOrderStatus
purchaseOrderLineItemNumber
purchaseOrderLineItemStatus
No change.
I should probably create a new issue to discuss further the remaining questions about the references to the sales order(s) and the purchase order(s) so we can close this issue with the commits containing all the changes we agreed on!
Remaining actions/discussions are described in issue #121 and issue #122.
2024-01-17, finally we should leave the purchaseOrderNumber
at the HEADER level and remove the salesOrderNumber
completely!
purchaseOrderNumber
is within the ListOfSupplierOrders.supplierOrders[]
(header-level).purchaseOrderNumber
is within the GetSupplierOrderById
(header-level).salesOrderNumber
is still within the GetSupplierOrderById.supplierOrderLineItems[]
(line-level). Let's remove it.Done in commit 56a0b2b.
numberOfLines: name? numberOfLineItems is used in ver. 2.0.0. Naming should be consistent. Name a bit confusing because both PurchaseOrderNumber and SalesOrderNumber exist and line Items of salesOrder might not be the same as for supplierOrder and purchaseOrder. Is it now clear that it is the numberOfLineItems of the supplierOrder only.
numberOfLines: optional? numberOfLineItems is mandatory in ListPurchaseOrders
supplierOrderStatus: optional? purchaseOrderStatus is required in ListPurchaseOrders
salesOrderNumber: is on purchaseOrderLineItem in ver. 2.00 Now in the header of the supplierOrder?