[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Owen Leonard changed: What|Removed |Added Keywords|release-notes-needed| -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Bug 38941 depends on bug 38930, which changed state. Bug 38930 Summary: Add a permissions store for Vue apps https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38930 What|Removed |Added Status|Needs documenting |RESOLVED Resolution|--- |FIXED -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 David Nind changed: What|Removed |Added CC||[email protected] Resolution|--- |FIXED Status|Needs documenting |RESOLVED --- Comment #28 from David Nind --- Architecture change, no change to the UI, no changes to the manual required. -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Bug 38941 depends on bug 37911, which changed state. Bug 37911 Summary: Prototype vue islands within static pages https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37911 What|Removed |Added Status|Needs documenting |RESOLVED Resolution|--- |FIXED -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 --- Comment #26 from Matt Blenkinsop --- > That said, can you explain things more? In the case of acquisitions menu, it > looks like you do have access to the pinia stores, but that it's just not > being populated? It also looks like it gets mounted as a single component > Vue app in the end? Looking at bug 38930, it looks like the permissionsStore > is not used anywhere currently? There's the ERM example which shows an > ERM-specific implementation for using the permissionsStore, but that seems > suboptimal. > > When I first heard of bug 38930, I thought it was going to introduce a core > API endpoint for fetching current user details and then put the permissions > in the permissionsStore for use by Vue components. Even outside of Vue, I > reckon having an API endpoint to fetch that information would be useful. Its currently being used by the Vendors Vue module - when the acquisitions menu is loaded as part of the vendors vue app we have access to the permissions store as it is populated by the module's config endpoint and the menu is just used as a generic vue component rather than a custom element. When it is used as a custom element, the permissionsStore is technically available but isn't populated as there is no specific vue app with a config endpoint to populate from. A future enhancement could certainly be to update the permissions store to self-populate and remove the need for the TT variables -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 --- Comment #24 from Matt Blenkinsop --- Hi David, the component that is used in the custom element can be used either as a Vue custom element in template toolkit or as a component in a Vue application. When used in a Vue app we get access to the pinia stores (permissionsStore) and can use the method provided by the permissionsStore to check permissions. Outside of a vue app however, we don't get access to a populated permissionsStore so we need to be able to accept the permissions from the template toolkit variables as component props -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 --- Comment #27 from David Cook --- (In reply to Matt Blenkinsop from comment #26) > Its currently being used by the Vendors Vue module - Ah I see it now. Both the dependency chain in Bugzilla and the import of ./Islands/AcquisitionsMenu.vue in koha-tmpl/intranet-tmpl/prog/js/vue/components/LeftMenu.vue. > when the acquisitions > menu is loaded as part of the vendors vue app we have access to the > permissions store as it is populated by the module's config endpoint and the > menu is just used as a generic vue component rather than a custom element. I see what you mean when looking at /cgi-bin/koha/acqui/acqui-home.pl vs /cgi-bin/koha/acquisition/vendors?supplier= > When it is used as a custom element, the permissionsStore is technically > available but isn't populated as there is no specific vue app with a config > endpoint to populate from. A future enhancement could certainly be to update > the permissions store to self-populate and remove the need for the TT > variables I reckon that would be a much more DRY option for sure. Anyway, cool beans. Thanks for explaining all that. -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 --- Comment #25 from David Cook --- (In reply to Matt Blenkinsop from comment #24) > the component that is used in the custom element can be used > either as a Vue custom element in template toolkit or as a component in a > Vue application. When used in a Vue app we get access to the pinia stores > (permissionsStore) and can use the method provided by the permissionsStore > to check permissions. Outside of a vue app however, we don't get access to a > populated permissionsStore so we need to be able to accept the permissions > from the template toolkit variables as component props Looking back at the patch again, I see that it does support both methodologies for all cases. On first read, I thought it was using one methodology in one place and the other methodology in other places. My bad! That said, can you explain things more? In the case of acquisitions menu, it looks like you do have access to the pinia stores, but that it's just not being populated? It also looks like it gets mounted as a single component Vue app in the end? Looking at bug 38930, it looks like the permissionsStore is not used anywhere currently? There's the ERM example which shows an ERM-specific implementation for using the permissionsStore, but that seems suboptimal. When I first heard of bug 38930, I thought it was going to introduce a core API endpoint for fetching current user details and then put the permissions in the permissionsStore for use by Vue components. Even outside of Vue, I reckon having an API endpoint to fetch that information would be useful. -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 --- Comment #23 from David Cook --- Little question... I notice this bug depends on bug 38930, but then we're using Template::Toolkit to set attributes mapped to permissions on the acquisitions-menu web component. Looking again at the template, it looks like some parts of the template use the permission store from bug 38930 and some use the attributes/properties for the web component? Seems like it's only half-implemented here? -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Fridolin Somers changed: What|Removed |Added Status|Pushed to main |Needs documenting CC||[email protected] ||m --- Comment #22 from Fridolin Somers --- Enhancement not pushed to 24.11.x -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Katrin Fischer changed: What|Removed |Added Keywords|additional_work_needed | -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 --- Comment #21 from Katrin Fischer --- Pushed for 25.05! Well done everyone, thank you! -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Katrin Fischer changed: What|Removed |Added Version(s)||25.05.00 released in|| Status|Passed QA |Pushed to main -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 --- Comment #20 from Katrin Fischer --- 1) - I can see now that this code is removed by 38010. So dropping my follow-up. -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941
Katrin Fischer changed:
What|Removed |Added
Keywords||additional_work_needed
--- Comment #19 from Katrin Fischer ---
1) QA script
FAIL koha-tmpl/intranet-tmpl/prog/en/includes/acquisitions-menu.inc
FAIL filters
missing_filter at line 25 ([% SET islands =
Asset.js("js/vue/dist/islands.esm.js").match('(src="([^"]+)")').1 %]
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Katrin Fischer changed: What|Removed |Added QA Contact|[email protected] |[email protected] |y.org | Keywords||release-notes-needed -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 --- Comment #18 from Jonathan Druart --- Created attachment 181726 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=181726&action=edit Bug 38941: Rebase for follow-up to bug 37911 Bug 37911 added a follow-up that removed the original entry point from rspack to use the esm.js files Signed-off-by: Michaela Sieber Signed-off-by: Jonathan Druart -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 --- Comment #17 from Jonathan Druart --- Created attachment 181725 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=181725&action=edit Bug 38941: Add a cypress test Signed-off-by: Michaela Sieber Signed-off-by: Jonathan Druart -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Jonathan Druart changed: What|Removed |Added Attachment #181627|0 |1 is obsolete|| Attachment #181629|0 |1 is obsolete|| Attachment #181711|0 |1 is obsolete|| --- Comment #16 from Jonathan Druart --- Created attachment 181724 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=181724&action=edit Bug 38941: Convert the acquisitions menu to a Vue web component This is an example patch for creating a Vue island component that can be used throughout Koha. The component can then also be used in the work in Bug 38010 to migrate Vendors to Vue. This removes duplication of code by having a .inc file as well as a vue component to maintain Test plan: 1) Apply all patches including dependencies 2) Bug 37911 requires a Vue upgrade so you will need to run yarn install, then run a yarn build 3) Navigate to the acquisitions home page 4) The left menu should appear as before 5) Switch on the EDIFACT syspref - the acq menu should now display the edifact related options 6) Try logging in as a new patron with reduced permissions, the menu should update correctly according to the permissions Signed-off-by: Michaela Sieber Signed-off-by: Jonathan Druart -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Jonathan Druart changed: What|Removed |Added Status|Signed Off |Passed QA -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Matt Blenkinsop changed: What|Removed |Added Attachment #181628|0 |1 is obsolete|| --- Comment #15 from Matt Blenkinsop --- Created attachment 181711 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=181711&action=edit Bug 38941: Add a cypress test -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 David Cook changed: What|Removed |Added CC||[email protected] -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Michaela Sieber changed: What|Removed |Added CC||[email protected] Status|Needs Signoff |Signed Off -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Biblibre Sandboxes changed: What|Removed |Added Attachment #181626|0 |1 is obsolete|| --- Comment #14 from Biblibre Sandboxes --- Created attachment 181629 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=181629&action=edit Bug 38941: Rebase for follow-up to bug 37911 Bug 37911 added a follow-up that removed the original entry point from rspack to use the esm.js files Signed-off-by: Michaela -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Biblibre Sandboxes changed: What|Removed |Added Attachment #181493|0 |1 is obsolete|| --- Comment #13 from Biblibre Sandboxes --- Created attachment 181628 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=181628&action=edit Bug 38941: Add a cypress test Signed-off-by: Michaela -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Biblibre Sandboxes changed: What|Removed |Added Attachment #181492|0 |1 is obsolete|| --- Comment #12 from Biblibre Sandboxes --- Created attachment 181627 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=181627&action=edit Bug 38941: Convert the acquisitions menu to a Vue web component This is an example patch for creating a Vue island component that can be used throughout Koha. The component can then also be used in the work in Bug 38010 to migrate Vendors to Vue. This removes duplication of code by having a .inc file as well as a vue component to maintain Test plan: 1) Apply all patches including dependencies 2) Bug 37911 requires a Vue upgrade so you will need to run yarn install, then run a yarn build 3) Navigate to the acquisitions home page 4) The left menu should appear as before 5) Switch on the EDIFACT syspref - the acq menu should now display the edifact related options 6) Try logging in as a new patron with reduced permissions, the menu should update correctly according to the permissions Signed-off-by: Michaela -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 --- Comment #11 from Matt Blenkinsop --- Created attachment 181626 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=181626&action=edit Bug 38941: Rebase for follow-up to bug 37911 Bug 37911 added a follow-up that removed the original entry point from rspack to use the esm.js files -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Matt Blenkinsop changed: What|Removed |Added Attachment #181491|0 |1 is obsolete|| --- Comment #10 from Matt Blenkinsop --- Created attachment 181493 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=181493&action=edit Bug 38941: Add a cypress test -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Matt Blenkinsop changed: What|Removed |Added Attachment #178243|0 |1 is obsolete|| --- Comment #9 from Matt Blenkinsop --- Created attachment 181492 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=181492&action=edit Bug 38941: Convert the acquisitions menu to a Vue web component This is an example patch for creating a Vue island component that can be used throughout Koha. The component can then also be used in the work in Bug 38010 to migrate Vendors to Vue. This removes duplication of code by having a .inc file as well as a vue component to maintain Test plan: 1) Apply all patches including dependencies 2) Bug 37911 requires a Vue upgrade so you will need to run yarn install, then run a yarn build 3) Navigate to the acquisitions home page 4) The left menu should appear as before 5) Switch on the EDIFACT syspref - the acq menu should now display the edifact related options 6) Try logging in as a new patron with reduced permissions, the menu should update correctly according to the permissions -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Matt Blenkinsop changed: What|Removed |Added Status|Patch doesn't apply |Needs Signoff -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Matt Blenkinsop changed: What|Removed |Added Attachment #178244|0 |1 is obsolete|| --- Comment #8 from Matt Blenkinsop --- Created attachment 181491 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=181491&action=edit Bug 38941: Add a cypress test -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Amaury GAU changed: What|Removed |Added CC||[email protected] -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 --- Comment #7 from Katrin Fischer --- Hi Janne, this bug has 2 dependencies that are not resolved yet. The sandboxes are not very clear when there is a conflict on applying the patches. Maybe have a look at the git log if you can. -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Janne Seppänen changed: What|Removed |Added CC||[email protected] Status|Needs Signoff |Patch doesn't apply --- Comment #6 from Janne Seppänen --- Tried testing on ByWater Solutions sandbox and on a local git build, but both end up with an error and the left menu doesn't appear at all. Seems like the code is looking for a module that's not there? Uncaught Error: Cannot find module '../stores/permissions' webpackMissingModule https://staff-bugiton.sandbox.bywatersolutions.com/intranet-tmpl/prog/js/vue/dist/islands_24.1200012.js:16487 https://staff-bugiton.sandbox.bywatersolutions.com/intranet-tmpl/prog/js/vue/dist/islands_24.1200012.js:16487 https://staff-bugiton.sandbox.bywatersolutions.com/intranet-tmpl/prog/js/vue/dist/islands_24.1200012.js:16774 islands_24.1200012.js:16487:50 -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Matt Blenkinsop changed: What|Removed |Added Attachment #176877|0 |1 is obsolete|| --- Comment #4 from Matt Blenkinsop --- Created attachment 178243 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178243&action=edit Bug 38941: Convert the acquisitions menu to a Vue web component This is an example patch for creating a Vue island component that can be used throughout Koha. The component can then also be used in the work in Bug 38010 to migrate Vendors to Vue. This removes duplication of code by having a .inc file as well as a vue component to maintain Test plan: 1) Apply all patches including dependencies 2) Bug 37911 requires a Vue upgrade so you will need to run yarn install, then run a yarn build 3) Navigate to the acquisitions home page 4) The left menu should appear as before 5) Switch on the EDIFACT syspref - the acq menu should now display the edifact related options 6) Try logging in as a new patron with reduced permissions, the menu should update correctly according to the permissions -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Matt Blenkinsop changed: What|Removed |Added Attachment #176878|0 |1 is obsolete|| --- Comment #5 from Matt Blenkinsop --- Created attachment 178244 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178244&action=edit Bug 38941: Add a cypress test -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Matt Blenkinsop changed: What|Removed |Added Blocks||38010 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38010 [Bug 38010] Migrate vendors to Vue -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Matt Blenkinsop changed: What|Removed |Added CC||[email protected], ||martin.renvoize@ptfs-europe ||.com, [email protected], ||[email protected], ||[email protected] ||m, [email protected] -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 --- Comment #3 from Matt Blenkinsop --- This is a POC - interested to hear any ideas for improvement. I considered using the NavigationItem component to render the links but it changes the HTML structure slightly and is largely dependent on the Vue router -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 --- Comment #2 from Matt Blenkinsop --- Created attachment 176878 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176878&action=edit Bug 38941: Add a cypress test -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 --- Comment #1 from Matt Blenkinsop --- Created attachment 176877 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176877&action=edit Bug 38941: Convert the acquisitions menu to a Vue web component This is an example patch for creating a Vue island component that can be used throughout Koha. The component can then also be used in the work in Bug 38010 to migrate Vendors to Vue. This removes duplication of code by having a .inc file as well as a vue component to maintain Test plan: 1) Apply all patches including dependencies 2) Bug 37911 requires a Vue upgrade so you will need to run yarn install, then run a yarn build 3) Navigate to the acquisitions home page 4) The left menu should appear as before 5) Switch on the EDIFACT syspref - the acq menu should now display the edifact related options 6) Try logging in as a new patron with reduced permissions, the menu should update correctly according to the permissions -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Matt Blenkinsop changed: What|Removed |Added Status|NEW |Needs Signoff -- 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Matt Blenkinsop changed: What|Removed |Added Assignee|[email protected] |matt.blenkinsop@ptfs-europe |ity.org |.com -- You are receiving this mail because: You are the assignee for the bug. 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/
[Koha-bugs] [Bug 38941] Convert the acquisitions menu to a Vue island
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38941 Matt Blenkinsop changed: What|Removed |Added Depends on||37911, 38930 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37911 [Bug 37911] Prototype vue islands within static pages https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38930 [Bug 38930] Add a permissions store for Vue apps -- 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/
