Bug 31417 - Re-instate the cataloguing sidebar menu
Summary: Re-instate the cataloguing sidebar menu
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Martin Renvoize
QA Contact: Katrin Fischer
URL:
Keywords:
Depends on: 31162
Blocks: 31556 31864
  Show dependency treegraph
 
Reported: 2022-08-19 07:51 UTC by Martin Renvoize
Modified: 2023-12-28 20:42 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Medium patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
22.11.00


Attachments
Bug 31417: Update and reference cat-menu.inc (23.36 KB, patch)
2022-08-19 11:20 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 31417: Update and reference cat-menu.inc (23.41 KB, patch)
2022-08-19 20:54 UTC, David Nind
Details | Diff | Splinter Review
Bug 31417: Update and reference cat-menu.inc (29.40 KB, patch)
2022-08-23 10:12 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 31417: Update and reference cat-menu.inc (31.25 KB, patch)
2022-10-03 11:21 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 31417: Update and reference cat-menu.inc (31.69 KB, patch)
2022-10-17 15:18 UTC, Martin Renvoize
Details | Diff | Splinter Review
Bug 31417: Update and reference cat-menu.inc (31.75 KB, patch)
2022-10-17 20:57 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Renvoize 2022-08-19 07:51:57 UTC
There's a minimal old include for a cataloging sidebar still in the codebase but it doesn't appear to be referenced anywhere now.

We should bring it up to date and in sync with the proposed cataloging homepage in bug 31162 and include it where appropriate.
Comment 1 Martin Renvoize 2022-08-19 11:20:42 UTC
Created attachment 139477 [details] [review]
Bug 31417: Update and reference cat-menu.inc

This patch reinstates the cat-menu.inc include and references it from
the various 'tools' that are closely tied to cataloging.

We use the new cataloging home page submission as a reference for which
options should appear in the menu.
Comment 2 David Nind 2022-08-19 20:54:49 UTC
Created attachment 139522 [details] [review]
Bug 31417: Update and reference cat-menu.inc

This patch reinstates the cat-menu.inc include and references it from
the various 'tools' that are closely tied to cataloging.

We use the new cataloging home page submission as a reference for which
options should appear in the menu.

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 David Nind 2022-08-19 20:56:35 UTC
I've signed off as is, but happy to do again once any changes (if required) from decisions for bug 31162 are incorporated.
Comment 4 Martin Renvoize 2022-08-23 10:12:46 UTC
Created attachment 139653 [details] [review]
Bug 31417: Update and reference cat-menu.inc

This patch reinstates the cat-menu.inc include and references it from
the various 'tools' that are closely tied to cataloging.

We use the new cataloging home page submission as a reference for which
options should appear in the menu.

Signed-off-by: David Nind <david@davidnind.com>
Comment 5 Martin Renvoize 2022-08-23 10:13:08 UTC
Minor rebase to keep up with bug 31162
Comment 6 Martin Renvoize 2022-10-03 11:21:54 UTC
Created attachment 141255 [details] [review]
Bug 31417: Update and reference cat-menu.inc

This patch reinstates the cat-menu.inc include and references it from
the various 'tools' that are closely tied to cataloging.

We use the new cataloging home page submission as a reference for which
options should appear in the menu.

Signed-off-by: David Nind <david@davidnind.com>
Comment 7 Martin Renvoize 2022-10-03 11:22:09 UTC
Another rebase
Comment 8 Martin Renvoize 2022-10-17 15:18:04 UTC
Created attachment 141994 [details] [review]
Bug 31417: Update and reference cat-menu.inc

This patch reinstates the cat-menu.inc include and references it from
the various 'tools' that are closely tied to cataloging.

We use the new cataloging home page submission as a reference for which
options should appear in the menu.

Signed-off-by: David Nind <david@davidnind.com>
Comment 9 Martin Renvoize 2022-10-17 15:18:32 UTC
Rebased and updated for final iteration of bug 31162
Comment 10 Katrin Fischer 2022-10-17 20:57:58 UTC
Created attachment 142030 [details] [review]
Bug 31417: Update and reference cat-menu.inc

This patch reinstates the cat-menu.inc include and references it from
the various 'tools' that are closely tied to cataloging.

We use the new cataloging home page submission as a reference for which
options should appear in the menu.

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 11 Tomás Cohen Arazi 2022-10-18 13:30:57 UTC
Pushed to master for 22.11.

Nice work everyone, thanks!