Requirements - MODORDSTOR-37 - Getting issue details... STATUS
Spike to determine whether the "first" and "last" fields in the collection schemas should be 0 or 1 based. Do we even need these fields? It doesn't look like other modules have them.
See https://github.com/folio-org/acq-models/pull/61#discussion_r244816230 for details
Investigation
- "first" and "last" fields are not being used in UI orders
- All APIs are setting "first" and "last" field in collection in mod-orders-storage but those are never being used further.
- In daily standup, this was brought up and consensus was these fields were created when repositories were defined but these fields were never used in production.
POC
Remove "first" and "last" field from example https://github.com/folio-org/acq-models/blob/master/mod-orders-storage/examples/adjustment_collection.sample
Remove "first" and "last" field from schema https://github.com/folio-org/acq-models/blob/master/mod-orders-storage/schemas/adjustment_collection.json
Remove reference to "first" and "last" field in storage module https://github.com/folio-org/mod-orders-storage/blob/master/src/main/java/org/folio/rest/impl/AdjustmentsAPI.java#L70-L77
mod-orders-storage build succeed and all tests pass
Solution Proposal
Remove "first" and "last" fields from:
- All schemas in mod-order-storage
- All examples in mod-order-storage
- All collection level API's in mod-orders-storage