Bug 35584

Summary: Missing licenses in about page
Product: Koha Reporter: Victor Grousset/tuxayo <victor>
Component: AboutAssignee: Owen Leonard <oleonard>
Status: Pushed to oldstable --- QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: david, fridolin.somers, phil
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:
24.05.00,23.11.02
Circulation function:
Bug Depends on: 34900    
Bug Blocks: 34623, 35633, 35638, 35640, 35642, 35643    
Attachments: Bug 35584: Add missing licenses to about page
Bug 35584: Add missing licenses to about page
Bug 35584: (follow-up) Add license information and fix sort order
Bug 35584: Add missing licenses to about page
Bug 35584: (follow-up) Add license information and fix sort order
Bug 35584: (QA Follow-up):
Bug 35584: Add missing licenses to about page
Bug 35584: (follow-up) Add license information and fix sort order
Bug 35584: (QA follow-up):
Bug 35584: (follow-up) Remove duplicate license entry

Description Victor Grousset/tuxayo 2023-12-15 20:33:57 UTC
Thanks David Nind for noticing it in Bug 34900

> I've just had a quick look (not comprehensive), and it looks like there are
> some software licences missing, including:
> - DataTables (koha-tmpl/intranet-tmpl/lib/datatables)
> - GreyBox (koha-tmpl/intranet-tmpl/lib/greybox)
> - Some jQuery plugins - jsTree, rowGroup (doesn't look the same as
> jquery.dataTables.grouping.js), fnReloadAjax (part of DataTables?),
> Humanized Messages, jquery.dataTables.columnFilter.js (part of DataTables?),
> highlight (part of DataTables?), jquery-jeditable
> (koha-tmpl/intranet-tmpl/lib/jquery/plugins/)
> - linters (several) (koha-tmpl/intranet-tmpl/lib/linters)
> - sortable (koha-tmpl/intranet-tmpl/lib/linters/sortable)
> - vis-timeline and vis-graph2d (koha-tmpl/intranet-tmpl/lib/vis-timeline/)
> - media.match.js (koha-tmpl/opac-tmpl/lib/)
Comment 1 Owen Leonard 2023-12-20 13:10:13 UTC
Created attachment 160116 [details] [review]
Bug 35584: Add missing licenses to about page

This patch adds license information to the about page which was missing
for several JavaScript libraries. The patch also makes a minor change to
CSS to improve alignment of content in the licenses tab.

One library, vis-timeline-graph2d.min.js, lacked the correct version
information in the source. This information has been added.

To test, apply the patch and rebuild the staff interface CSS.

- Go to the about page and open the "Licenses" tab.
- Confirm that the information looks correct.
Comment 2 David Nind 2023-12-20 20:16:49 UTC
Created attachment 160162 [details] [review]
Bug 35584: Add missing licenses to about page

This patch adds license information to the about page which was missing
for several JavaScript libraries. The patch also makes a minor change to
CSS to improve alignment of content in the licenses tab.

One library, vis-timeline-graph2d.min.js, lacked the correct version
information in the source. This information has been added.

To test, apply the patch and rebuild the staff interface CSS.

- Go to the about page and open the "Licenses" tab.
- Confirm that the information looks correct.

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 David Nind 2023-12-20 20:16:51 UTC
Created attachment 160163 [details] [review]
Bug 35584: (follow-up) Add license information and fix sort order

Add license information and links. Put in alphabetical order.

Signed-off-by: David Nind <david@davidnind.com>
Comment 4 David Nind 2023-12-20 20:17:30 UTC
Thanks Owen!
Comment 5 Pedro Amorim 2023-12-22 12:37:23 UTC
Created attachment 160231 [details] [review]
Bug 35584: Add missing licenses to about page

This patch adds license information to the about page which was missing
for several JavaScript libraries. The patch also makes a minor change to
CSS to improve alignment of content in the licenses tab.

One library, vis-timeline-graph2d.min.js, lacked the correct version
information in the source. This information has been added.

To test, apply the patch and rebuild the staff interface CSS.

- Go to the about page and open the "Licenses" tab.
- Confirm that the information looks correct.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Comment 6 Pedro Amorim 2023-12-22 12:37:25 UTC
Created attachment 160232 [details] [review]
Bug 35584: (follow-up) Add license information and fix sort order

Add license information and links. Put in alphabetical order.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Comment 7 Pedro Amorim 2023-12-22 12:37:28 UTC
Created attachment 160233 [details] [review]
Bug 35584: (QA Follow-up):

Typo fix: GreBox -> Greybox
Capitalization fix: JavaScript case
Order to match header fix: jsTree jQuery -> jQuery jsTree
Capitalization fix: treetable -> Treetable capitalization

Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Comment 8 Katrin Fischer 2023-12-27 16:33:00 UTC
The dependency here is still "NSO" and the last patch here should have a proper description please :)
Comment 9 Jonathan Druart 2024-01-04 12:43:15 UTC
Created attachment 160526 [details] [review]
Bug 35584: Add missing licenses to about page

This patch adds license information to the about page which was missing
for several JavaScript libraries. The patch also makes a minor change to
CSS to improve alignment of content in the licenses tab.

One library, vis-timeline-graph2d.min.js, lacked the correct version
information in the source. This information has been added.

To test, apply the patch and rebuild the staff interface CSS.

- Go to the about page and open the "Licenses" tab.
- Confirm that the information looks correct.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Comment 10 Jonathan Druart 2024-01-04 12:43:17 UTC
Created attachment 160527 [details] [review]
Bug 35584: (follow-up) Add license information and fix sort order

Add license information and links. Put in alphabetical order.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Comment 11 Jonathan Druart 2024-01-04 12:43:20 UTC
Created attachment 160528 [details] [review]
Bug 35584: (QA follow-up):

Typo fix: GreBox -> Greybox
Capitalization fix: JavaScript case
Order to match header fix: jsTree jQuery -> jQuery jsTree
Capitalization fix: treetable -> Treetable capitalization

Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Comment 12 Katrin Fischer 2024-01-04 15:44:27 UTC
It looks like a double up:

jQuery UI Virtual Keyboard
Virtual Keyboard

Same version and link?
Comment 13 Owen Leonard 2024-01-08 17:02:15 UTC
Created attachment 160653 [details] [review]
Bug 35584: (follow-up) Remove duplicate license entry
Comment 14 Katrin Fischer 2024-01-11 11:12:03 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 15 Fridolin Somers 2024-01-15 14:02:17 UTC
Pushed to 23.11.x for 23.11.02