[Bug 43161] New: ILL module should have a /config endpoint
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43161 Bug ID: 43161 Summary: ILL module should have a /config endpoint Initiative type: --- Sponsorship --- status: Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: jonathan.druart@gmail.com Reporter: jonathan.druart@gmail.com QA Contact: testopia@bugs.koha-community.org Target Milestone: --- To retrieve the sysprefs. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43161 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |43162 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43162 [Bug 43162] Cypress tests for the ILL module should mock the /config endpoint -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43161 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43161 --- Comment #1 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 202463 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=202463&action=edit Bug 43161: Add a /config endpoint for the ILL Vue app The other Vue app (ERM, Preservation) use a /config endpoint for the syspref. The ILL Vue app should also have one. Test plan: 0. as kohadev-koha: `yarn api:bungle && yarn js:build` as root: `koha-plack --reload kohadev` 1. Turn off the ILLModule syspref 2. Go to /cgi-bin/koha/ill/home => The app is available 3. Turn off the syspref => The app is disabled -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43161 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |41995 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41995 [Bug 41995] Autocomplete for syspref search -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43161 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #202463|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43161 --- Comment #2 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 202464 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=202464&action=edit Bug 43161: Add a /config endpoint for the ILL Vue app The other Vue app (ERM, Preservation) use a /config endpoint for the syspref. The ILL Vue app should also have one. Test plan: 0. as kohadev-koha: `yarn api:bungle && yarn js:build` as root: `koha-plack --reload kohadev` 1. Turn off the ILLModule syspref 2. Go to /cgi-bin/koha/ill/home => The app is available 3. Turn off the syspref => The app is disabled -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43161 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43161 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #202464|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43161 --- Comment #3 from David Nind <david@davidnind.com> --- Created attachment 202568 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=202568&action=edit Bug 43161: Add a /config endpoint for the ILL Vue app The other Vue app (ERM, Preservation) use a /config endpoint for the syspref. The ILL Vue app should also have one. Test plan: 0. as kohadev-koha: `yarn api:bungle && yarn js:build` as root: `koha-plack --reload kohadev` 1. Turn on the ILLModule syspref 2. Go to /cgi-bin/koha/ill/home => The app is available 3. Turn off the syspref => The app is disabled Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43161 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com --- Comment #4 from David Nind <david@davidnind.com> --- I amended the test plan in the commit message slightly: 1. Step 1 said to turn off the ILL module, I think this was supposed to say enable the ILL module. Testing notes (using KTD): 1. Step 0: - Rebuild everything: yarn build - Restart everything: restart_all -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43161 Pedro Amorim (ammopt) <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |RESOLVED Resolution|--- |DUPLICATE CC| |pedro.amorim@openfifth.co.u | |k --- Comment #5 from Pedro Amorim (ammopt) <pedro.amorim@openfifth.co.uk> --- Hi Joubu, I'm marking this as duplicate of bug 42845 as that bug happened first and contains tests. Are you available to help QA there? Thanks in advance. *** This bug has been marked as a duplicate of bug 42845 *** -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org