Librarians would like the page to be automatically refreshed every 5 minutes. I am wondering if we need: * a syspref * hardcode 5 minutes (certainly not) * provide JS to include in IntranetUserJS, but don't provide the feature out of the box
Katrin, what's your opinion on this one?
Maybe providing JS if possible is a good idea to get more feedback - I am not sure actually if it would be a good feature or not. I an ideal setting we should probably trigger a refresh when another pickup was scheduled.
Created attachment 143237 [details] [review] Bug 30963: Automatically refresh the curbside pickups list Librarians would like the page to be automatically refreshed every X minutes. This patch is suggesting to have a checkbox at the top of the page to let the user refresh the page given the delay they want. We could later improve this by setting this value in a localStorage variable. Test plan: Go to the curbside pickup page Notice the new checkbox at the top Notice that the "Refresh" button has been moved at the top as well Play with the checkbox and the input, and confirm that the behaviour is consistent Sponsored-by: Association KohaLa - https://koha-fr.org/
I went with the nice version, I think (not the ideal one however). Note that I won't have much additional time for this. https://snipboard.io/sfo6We.jpg
Created attachment 143295 [details] [review] Bug 30963: Automatically refresh the curbside pickups list Librarians would like the page to be automatically refreshed every X minutes. This patch is suggesting to have a checkbox at the top of the page to let the user refresh the page given the delay they want. We could later improve this by setting this value in a localStorage variable. Test plan: Go to the curbside pickup page Notice the new checkbox at the top Notice that the "Refresh" button has been moved at the top as well Play with the checkbox and the input, and confirm that the behaviour is consistent Sponsored-by: Association KohaLa - https://koha-fr.org/ Signed-off-by: David Nind <david@davidnind.com>
Created attachment 144073 [details] [review] Bug 30963: Automatically refresh the curbside pickups list Librarians would like the page to be automatically refreshed every X minutes. This patch is suggesting to have a checkbox at the top of the page to let the user refresh the page given the delay they want. We could later improve this by setting this value in a localStorage variable. Test plan: Go to the curbside pickup page Notice the new checkbox at the top Notice that the "Refresh" button has been moved at the top as well Play with the checkbox and the input, and confirm that the behaviour is consistent Sponsored-by: Association KohaLa - https://koha-fr.org/ Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Pushed to master for 23.05. Nice work everyone, thanks!
ENHANCEMENT: Won't be back-ported to 22.11.x Apologies!