http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7791
M. de Rooy <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA Patch complexity|Small patch |Medium patch Severity|critical |normal --- Comment #38 from M. de Rooy <[email protected]> --- QA Comment: Works as advertised. Nice to have. No complaints from qa tools. Code looks good to me. Includes string changes, so candidate for 3.16. I am passing QA on these patches, but have some hesitations as to design and navigation, as described in the following points: 1) The design of the page showing that the basket has been deleted, but also listing a warning c/should look better (layman's perspective :) 2) This same error page gives me the link Go back to booksellers page (note the problems reported earlier about its performance). Since I came from a basket of a specific bookseller, I would rather go back to the basket list of THAT bookseller. You opened up a new report for that, I understand. I think that this set of patches would be even more attractive with such a link. 3) A successful deletion actually does the same and sends me back to all booksellers. Please give me my particular bookseller instead :) Thanks. PS I downgraded the bug severity. I can understand that you feel this is an important issue, but critical is really another category. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] http://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/
