http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6403

Paul Poulain <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |[email protected]
       Patch Status|Signed Off                  |Passed QA

--- Comment #6 from Paul Poulain <[email protected]> 2011-10-07 
14:52:37 UTC ---
QA comment
  * good test plan
  * works as expected
  * nothing to say about the code itself

My only comment could be that some libraries consider saving such returns as a
bug, so we could set the default value of the new syspref to 1 directly.
But we've a rule to say a new syspref sould not change the behaviour for
libraries that don't do anything with it.

SO :
* passed QA
* it's worth being very clear in the release note that most libraries will
probably want to set this value to 1 after upgrading

-- 
Configure bugmail: 
http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
_______________________________________________
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/

Reply via email to