[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