https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25534 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Failed QA CC| |jonathan.druart@bugs.koha-c | |ommunity.org --- Comment #70 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- 1. Why do we have the dropdown list with the reason list at the bottom of this table: https://snipboard.io/i0RrM2.jpg Would not it make sense to have it in the confirmation popup (moved to a modal)? 2. Should not we have the cancellation reason displayed in the "Holds history" view? /cgi-bin/koha/members/holdshistory.pl?borrowernumber=51 3. POD for Koha::Hold->cancel needs to be adjusted 4. de-DE/optional/auth_val.sql contains twice the same insert statements 5. Tests must use build_sample_item (see bug 26250) I will attach a follow-up for 5. -- You are receiving this mail because: You are watching all bug changes.