[Koha-bugs] [Bug 25965] Create SIP2 client daemon with HTTPS API endpoint
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Fri Sep 18 01:18:23 CEST 2020
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25965
--- Comment #10 from David Cook <dcook at prosentient.com.au> ---
I notice a few hard-coded references to koha-suomi.fi, so those would need to
be removed and use replaceable placeholders.
You'll want to take a look at the coding guidelines too
(https://wiki.koha-community.org/wiki/Coding_Guidelines), as I think you've
used tabs here instead of 4 spaces (per tab).
For the config, you might want to put it into koha-conf.xml as well (even if I
do think that we overload that 1 configuration file with everything...)
Other than that, can you give us a test plan for how to get this to work?
Ideally, it would be great if it could work in koha-testing-docker
(https://gitlab.com/koha-community/koha-testing-docker).
Overall, it's looking good. Thanks for posting it!
--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
More information about the Koha-bugs
mailing list