http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5528
Ian Walls <ian.wa...@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch Status|Signed Off |Passed QA --- Comment #17 from Ian Walls <ian.wa...@bywatersolutions.com> 2011-10-12 15:54:47 UTC --- Thank you, Jared, for incorporating the necessary changes and attaching as a patch. The detailed commit message is MUCH appreciated, as it provides clear documentation on what this feature does and how. Reviewing the code, I see that things are rewritten very modularly, making minimal changes to existing functionality (and in those cases usually wrapping the changes in conditional checks). I think we're pretty well safe from regressions. I'd like to see the database table connections, and the 773 links be bibliolevel instead of item level, but those are just my preferences, and do not actually affect the functionality of this development. Marking as Passed QA. Chris, would it be possible for you to add in the Author credits in Git so that everyone who has worked on this squashed patch can get the proper credit? -- 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 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/