]> git.koha-community.org Git - koha.git/commit
Bug 32983: ERM - Retrieve AVs from an endpoint
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Thu, 16 Feb 2023 13:38:46 +0000 (14:38 +0100)
committerTomas Cohen Arazi <tomascohen@theke.io>
Thu, 2 Mar 2023 17:46:09 +0000 (14:46 -0300)
commit5d8c6a36b8647aa33945e4e6e9b28e511cd433bb
tree334f947aea5e21326d691c0764a4c4ca328792d3
parent611a77ca515111296a461336ddcfc50e77a25b8e
Bug 32983: ERM - Retrieve AVs from an endpoint

Bug 32981 let us retrieve the authorised values from a REST API route,
instead of injecting them from the template. Let us that for the ERM module!

Test plan:
You will notice a "Loading" screen when refreshing the ERM module
Then you should not notice any other UI changes. Dropdown list should be
populated like before this patch.

Some technical notes:
I am expecting this to be slower than before, but it feels better to use
a REST API route to retrieve the AV
Future improvement will be to lazy load the AVs, to speed up the landing
page. However it needs more changes, and this gets big enough.
I would like to see a follow-up that move the code from ERM/Main.vue to
the authorised value store (I've failed at that), but that should
certainly be done after the lazy loading is implemented anyway)

Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
19 files changed:
koha-tmpl/intranet-tmpl/prog/en/modules/erm/erm.tt
koha-tmpl/intranet-tmpl/prog/js/vue/components/Dialog.vue
koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementLicenses.vue
koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementRelationships.vue
koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementsFormAdd.vue
koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsEBSCOPackageTitlesList.vue
koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsEBSCOTitlesList.vue
koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalPackagesFormAdd.vue
koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalTitlesFormAdd.vue
koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/LicensesFormAdd.vue
koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/Main.vue
koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/UserRoles.vue
koha-tmpl/intranet-tmpl/prog/js/vue/fetch/api-client.js
koha-tmpl/intranet-tmpl/prog/js/vue/fetch/authorised-values.js [new file with mode: 0644]
koha-tmpl/intranet-tmpl/prog/js/vue/messages.js
koha-tmpl/intranet-tmpl/prog/js/vue/modules/erm.ts
koha-tmpl/intranet-tmpl/prog/js/vue/stores/authorised-values.js [new file with mode: 0644]
koha-tmpl/intranet-tmpl/prog/js/vue/stores/authorised_values.js [deleted file]
koha-tmpl/intranet-tmpl/prog/js/vue/stores/main.js