https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22417
Kyle M Hall <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|[email protected] |[email protected] |y.org | Status|Signed Off |Failed QA --- Comment #148 from Kyle M Hall <[email protected]> --- This looks really great! There are some rough edges that will need polished, but that doesn't necessarily need to happen right now on this bug. A few notes: * Why are koha_worker.pl and new_koha_job.pl in root dir? * Why does new_koha_job.pl use Koha::BackgroundJob::BatchUpdateBiblio but not Koha::BackgroundJob::BatchUpdateAuthority? Is this needed * use Scalar::Util qw( weaken ); not used I think at this point, the focus could be on cleaning up the ui and not refactoring the internals until they need to be refactored. The most glaring issue is this message presented to the user: The job has been enqueued! It will be processed as soon as possible (FIXME - well, it could depend on a config?) -- 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/
