Bug 37705 - Use REST API for patron_holds_table ( reserve/request.pl )
Summary: Use REST API for patron_holds_table ( reserve/request.pl )
Status: RESOLVED DUPLICATE of bug 23269
Alias: None
Product: Koha
Classification: Unclassified
Component: Hold requests (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Pedro Amorim
QA Contact: Testopia
URL: https://github.com/PTFS-Europe/koha/t...
Keywords:
Depends on:
Blocks: 34832
  Show dependency treegraph
 
Reported: 2024-08-21 21:05 UTC by Lucas Gass (lukeg)
Modified: 2024-10-21 08:22 UTC (History)
7 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.