https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41310

--- Comment #14 from David Cook <[email protected]> ---
The test plan is missing the steps for upgrading the database and restarting
services but that's ok. Not a blocker.

I don't know about having "ActionLogsEnableIPLogging" under "Debugging" instead
of "Logging", but... I'm not too fussed. If we want to change the description
and categorisation later, it's not hard to do, so I think it's fine.

I can't recall if we're supposed to include Koha/Schema/Result/ActionLog.pm in
the main patch. I think it's supposed to be in a separate patch, but I'm
double-checking with QA team on that one.

Other than that... I think it's all good. It works as described. I'll mark as
Signed Off for now.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://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