http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11708
--- Comment #86 from M. de Rooy <m.de.r...@rijksmuseum.nl> --- QA Comment: Passes qa tools. Verified Basketgroups.t. This test does not contain any testnames in the calls to ok, is, etc. It is encouraged to do so. Could you please add? Question: Do we still need the column closed, if you add a closeddate in aqbasketgroups? Do we create redundancy? And what about confusion like: Hey closed=0 but there is a closeddate? Please convince me that we should have both columns. [PATCH 6/7] Bug 11708: Remove the amount columns Commit message says only: This should be managed in a specific bug. Please explain. Before continuing this QA session, I would appreciate feedback on the points listed. Thanks! -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/