[Koha-bugs] [Bug 30650] Convert the Curbside Pickup plugin to Koha core

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Fri Jul 29 12:00:48 CEST 2022


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

--- Comment #107 from Katrin Fischer <katrin.fischer at bsz-bw.de> ---
o) Tests are failing, but possibly just a typo/later change causing it:

    #   Failed test 'Notice correctly generated'
    #   at t/db_dependent/Koha/CurbsidePickups.t line 299.
    #          got: 'You have schedule a curbside pickup for Te9oDc'
    #     expected: 'You have schedule a curbside pickup for Te9oDc.'
    # Looks like you failed 1 test of 2.


Other finds from code review:

1) Tiny typos

* Curbside pickups are not enabled for your location. = library
* Pickup Date/Time = capitalization
* ... it is an holiday = it is a holiday. (multiple)
* Deliver Date/Time = capitalization
* There is no waiting holds for this patron at this location. = There are no...
at this library
* No pickup time define for this day. = defined (multiple)
* You don't have waiting holds at this location = library

2) New JS libraries

There are no license statements for these, as we only add the minified version:

dayjs.min.js
isSameOrAfter.js
customParseFormat.js

Should they be listed on the about page?

Please check!

I'll keep testing on GUI side.

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list