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

--- Comment #3 from Katrin Fischer <[email protected]> ---
I've rebased the patches and done a first code review. I like the concept of
this as it's small and flexible. 

I wonder, if you are still around for this Magnus? I can spend some time on ILL
developments right now as we are looking into a backend for our ILL workflow
and this would help a great deal.

Looking at the code it seems like providing no due date in the ill request
would be problematic. I'd suggest to check if the date is set and then have a
fallback to normal due_date calculation if it isn't. 

There is also a leftover warn.

I can also sign-off, but I think for QA we'd need tests...

-- 
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