Bug 32998

Summary: Consolidate opac-tmpl/lib and opac-tmpl/bootstrap/lib
Product: Koha Reporter: Owen Leonard <oleonard>
Component: OPACAssignee: Owen Leonard <oleonard>
Status: RESOLVED FIXED QA Contact: Katrin Fischer <katrin.fischer>
Severity: enhancement    
Priority: P5 - low CC: matt.blenkinsop
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.05.00
Bug Depends on: 32969    
Bug Blocks: 34093    
Attachments: Bug 32998: Consolidate opac-tmpl/lib and opac-tmpl/bootstrap/lib
Bug 32998: Consolidate opac-tmpl/lib and opac-tmpl/bootstrap/lib
Bug 32998: Consolidate opac-tmpl/lib and opac-tmpl/bootstrap/lib

Description Owen Leonard 2023-02-17 15:39:58 UTC
We have both opac-tmpl/lib and opac-tmpl/bootstrap/lib. Each one has various third-party assets which are used in the OPAC. Since these assets do not need to be in a place where the translation tools can access them we should put them together in opac-tmpl/lib.
Comment 1 Owen Leonard 2023-02-17 17:22:48 UTC
Created attachment 146897 [details] [review]
Bug 32998: Consolidate opac-tmpl/lib and opac-tmpl/bootstrap/lib

This patch moves OPAC third-party assets from opac-tmpl/bootstrap/lib
to opac-tmpl/lib, eliminating some redundancy and making it simpler
to understand where such assets should be found.

To test, apply the patch and confirm that there is no longer a lib
directory under opac-tmpl/bootstrap.

Test various parts of the OPAC to confirm that assets are still loading
correctly. Everything should look right and work correctly. The
browser's developer console should be free of errors.
Comment 2 Philip Orr 2023-03-29 11:28:14 UTC
Created attachment 148902 [details] [review]
Bug 32998: Consolidate opac-tmpl/lib and opac-tmpl/bootstrap/lib

This patch moves OPAC third-party assets from opac-tmpl/bootstrap/lib
to opac-tmpl/lib, eliminating some redundancy and making it simpler
to understand where such assets should be found.

To test, apply the patch and confirm that there is no longer a lib
directory under opac-tmpl/bootstrap.

Test various parts of the OPAC to confirm that assets are still loading
correctly. Everything should look right and work correctly. The
browser's developer console should be free of errors.

Signed-off-by: Philip Orr <philip.orr@lmscloud.de>
Comment 3 Katrin Fischer 2023-04-09 11:24:51 UTC
Created attachment 149349 [details] [review]
Bug 32998: Consolidate opac-tmpl/lib and opac-tmpl/bootstrap/lib

This patch moves OPAC third-party assets from opac-tmpl/bootstrap/lib
to opac-tmpl/lib, eliminating some redundancy and making it simpler
to understand where such assets should be found.

To test, apply the patch and confirm that there is no longer a lib
directory under opac-tmpl/bootstrap.

Test various parts of the OPAC to confirm that assets are still loading
correctly. Everything should look right and work correctly. The
browser's developer console should be free of errors.

Signed-off-by: Philip Orr <philip.orr@lmscloud.de>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Tomás Cohen Arazi 2023-04-13 14:49:19 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 5 Matt Blenkinsop 2023-04-17 09:15:35 UTC
Enhancement - not backporting to 22.11.x

Nice work everyone!