Bug 37705

Summary: Use REST API for patron_holds_table ( reserve/request.pl )
Product: Koha Reporter: Lucas Gass (lukeg) <lucas>
Component: Hold requestsAssignee: Pedro Amorim <pedro.amorim>
Status: RESOLVED DUPLICATE QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: andrew.auld, george, gmcharlt, johanna.raisa, lisette, martin.renvoize, pedro.amorim
Version: Main   
Hardware: All   
OS: All   
URL: https://github.com/PTFS-Europe/koha/tree/bug_37705
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on:    
Bug Blocks: 34832    

Description Lucas Gass (lukeg) 2024-08-21 21:05:30 UTC
With a large amount of holds the patron_holds_table ( reserve/request.pl ) is very slow to load. 

REST API + pagination would help solve this.
Comment 1 George Williams (NEKLS) 2024-09-04 16:15:31 UTC
Good idea
Comment 2 Pedro Amorim 2024-09-09 14:07:40 UTC
Hi guys, I'm doing work here and plan to submit something in the next few days.
Comment 3 Pedro Amorim 2024-10-18 10:13:44 UTC
Thank you Nick for reminding me of this. I had done progress on it in the past but had since moved away from it.
I've pushed my current WIP to the branch URL in the bug. I still intend to resume this but it's not my top priority right now, unfortunately.
I remember finding the "change priority" functionality particularly challenging to implement on the API based table, but thinking about it again it should, in theory, be a case of redrawing the datatable every time some priority is updated, which is the behavior of the client-side table currently I believe i.e. it's always sorted by priority ascending.
Comment 4 Johanna Räisä 2024-10-21 05:40:47 UTC
I'm happy to help with this. There is a duplicate bug about it and it is assigned to me. I have made few depended bugs so things can be updated with API, 
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30661 and https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30660

*** This bug has been marked as a duplicate of bug 23269 ***
Comment 5 Pedro Amorim 2024-10-21 08:22:12 UTC
Thank you Johanna, the bugs you linked look great.
I'll try to make some time to go through these to help SO/QA.