Bug 18731 - CRUD for acquisition orders
Summary: CRUD for acquisition orders
Status: Needs Signoff
Alias: None
Product: Koha
Classification: Unclassified
Component: REST api (show other bugs)
Version: master
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Tomás Cohen Arazi
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 19196 19234 19369 19370 19410 18137 19209
Blocks:
  Show dependency treegraph
 
Reported: 2017-06-05 16:46 UTC by Kyle M Hall
Modified: 2017-12-05 18:38 UTC (History)
1 user (show)

See Also:
Change sponsored?: Sponsored
Patch complexity: ---
Bot Control: ---
When did the bot last check this:
Who signed the patch off:
Text to go in the release notes:


Attachments
Bug 18731: WIP /acquisitions/orders endpoint (5.25 KB, patch)
2017-11-16 18:52 UTC, Tomás Cohen Arazi
Details | Diff | Splinter Review
Bug 18731: OpenAPI definitions (18.95 KB, patch)
2017-11-16 18:52 UTC, Tomás Cohen Arazi
Details | Diff | Splinter Review
Bug 18731: /acquisitions/orders endpoint (5.91 KB, patch)
2017-11-23 14:18 UTC, Tomás Cohen Arazi
Details | Diff | Splinter Review
Bug 18731: OpenAPI definitions (19.68 KB, patch)
2017-11-23 14:18 UTC, Tomás Cohen Arazi
Details | Diff | Splinter Review
Bug 18731: OpenAPI definitions (18.75 KB, patch)
2017-12-05 18:38 UTC, Tomás Cohen Arazi
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Kyle M Hall 2017-06-05 16:46:48 UTC
Some libraries may have hundreds of open orders for a vendor, depending on acqisitions workflow. At that number of open orders, the tables on parcel.pl take 10 to 20 seconds to load.

To speed up the loading of this page and the data, we should convert the order tables from pre-loading all order data to ajax tables that load order data on an as needed basis. That way we are only pulling the data for the orders that are currently visible.
Comment 1 Katrin Fischer 2017-08-15 21:04:11 UTC
Was a bit confused by the bug title - hope I made it a bit more clear.
Comment 2 Tomás Cohen Arazi 2017-11-16 18:52:52 UTC
Created attachment 69184 [details] [review]
Bug 18731: WIP /acquisitions/orders endpoint

This patches implement the /acquisitions/orders endpoint.
It uses the new pagination and query parsing plugins.

Sponsored-by: Camden County
Comment 3 Tomás Cohen Arazi 2017-11-16 18:52:56 UTC
Created attachment 69185 [details] [review]
Bug 18731: OpenAPI definitions

This patch introduces the OpenAPI definitions required for the orders
endpoint. It adds query parameters definitions for the reserved params:

- _match
- _order_by
- _page
- _per_page
Comment 4 Tomás Cohen Arazi 2017-11-23 14:18:35 UTC
Created attachment 69309 [details] [review]
Bug 18731: /acquisitions/orders endpoint

This patches implement the /acquisitions/orders endpoint.
It uses the new pagination, query parsing and object searching plugins.

Sponsored-by: Camden County
Comment 5 Tomás Cohen Arazi 2017-11-23 14:18:42 UTC
Created attachment 69310 [details] [review]
Bug 18731: OpenAPI definitions

This patch introduces the OpenAPI definitions required for the orders
endpoint. It adds query parameters definitions for the reserved params:

- _match
- _order_by
- _page
- _per_page
Comment 6 Tomás Cohen Arazi 2017-12-05 18:38:30 UTC
Created attachment 69544 [details] [review]
Bug 18731: OpenAPI definitions

This patch introduces the OpenAPI definitions required for the orders
endpoint.