https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41920 Pedro Amorim (ammopt) <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Failed QA --- Comment #18 from Pedro Amorim (ammopt) <pedro.amorim@openfifth.co.uk> --- Hey, a few things here: 1) The 2nd patch appears to have been wrongly rebased (It's duplicating code). Please double-check. 2) Test coverage for total_running_reports_per_instance_limit would be ideal, not only because it's a new config but also because its siblings total_running_reports_per_user_limit and duplicate_running_reports_per_user_limit already have it. 3) $instance_id variable declared but never used in guided_reports.pl -- You are receiving this mail because: You are watching all bug changes.