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

--- Comment #6 from Matt Blenkinsop <[email protected]> ---
Created attachment 171628
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=171628&action=edit
Bug 37930: [DO NOT PUSH] Alternate concept

Alternative patch taking into account feedback in comment 5 on the bug. This
patch removes the authorised values store and migrates the methods to the main
store. Authorised values are now added into the module specific store and
replaced on module load with the actual values

To test I've added a console log of the values in the store to the
AgreementsList component so apply all patches on the bug and then navigate to
the agreements table in ERM. The module will throw errors as I haven't been
through and changed all the instances where we would need to swap the AV store
for the ERM store (there are a lot of instances and this is just a proof of
concept for discussion for now). Check that the logged values are as expected.
If you have the Vue devtools you'll be able to see the store values and check
them directly in there

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to