https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135
Emmi Takkinen <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |ASSIGNED --- Comment #32 from Emmi Takkinen <[email protected]> --- (In reply to Nick Clemens (kidclamp) from comment #31) > I like this a lot - self contained and makes minimal changes, the patches > are clean and easy to read, but I think we need to consider some things here > before this moves ahead > > Must fix: > The JS alerts to prevent submission fire an alert per warning, so you need > to click through several if you make a mistake - these should be combined > into a single alert > > There is no way to see the limits you have placed on your search - I think > these need to be shown to the user, ideally they would be editable, but > showing them would be a start > > Should fix: > While I like the idea of minimalizing data for privacy, excluding the > borrower info on the holds feels a bit too far. Ideally this would be under > columns configuration, and we have patron-title.inc for showing borrower > info only to staff who should see it. The current column selection feels > > The display should be paginated, ideally it would use the holds API to load > the holds to ensure performance, as scrolling can get really long really > fast. It would be nice too to be able to select all in a filtered search - > right now 'Select all' selects even those not visible in the table > > Could fix: > If implemented with the datatables API this could then work like the patron > search, with filters present on the page. > > The batch edit implies more batch actions - cancellation, reverting, etc. > but these can all be follow-ups I think Thank you Nick for feedback. First two points are indeed something that should be fixed. I'm just bit unsure how to display those search terms in search result page. I also agree that using API does have benefits and table should be paginated (even my test data might produce result of over 2000 holds and that is cumbersome for users go through). I'll take a look what I can do to these. Setting this as assigned for now. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
