Bug 33356

Summary: Add link to mana-kb settings from 'Useful resources' in reports
Product: Koha Reporter: Martin Renvoize (ashimema) <martin.renvoize>
Component: Mana-kbAssignee: Martin Renvoize (ashimema) <martin.renvoize>
Status: Needs documenting --- QA Contact:
Severity: trivial    
Priority: P5 - low CC: alex.arnaud, david, matt.blenkinsop, oleonard, paul.poulain
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.05.00
Circulation function:
Attachments: Bug 33356: Highlight mana-kb from reports when not configured
Bug 33356: Highlight mana-kb from reports when not configured
Bug 33356: [Alternate) Add mana-kb hint to reports start page
Bug 33356: Add mana-kb hint to reports start page
Bug 33356: Add mana-kb hint to reports start page

Description Martin Renvoize (ashimema) 2023-03-29 13:21:56 UTC
When mana-kb is not yet enabled, we should highlight that it exists to people writing reports so they know to enable it.
Comment 1 Martin Renvoize (ashimema) 2023-03-29 13:40:19 UTC
Created attachment 148917 [details] [review]
Bug 33356: Highlight mana-kb from reports when not configured

This patch adds a 'Knowledgebase' info block under 'Useful resources' on
teh report creation pages. This highlights to reports users that mana
has not be configured as yet, hopefully encouraging them to turn it on.
Comment 2 David Nind 2023-04-01 20:50:15 UTC
Created attachment 149067 [details] [review]
Bug 33356: Highlight mana-kb from reports when not configured

This patch adds a 'Knowledgebase' info block under 'Useful resources' on
teh report creation pages. This highlights to reports users that mana
has not be configured as yet, hopefully encouraging them to turn it on.

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 David Nind 2023-04-01 20:53:04 UTC
Testing notes (using KTD):

1. To see the highlight in the left sidebar, select any of the report creation links from the reports page (such as create guided report, create from SQL, and use saved).
Comment 4 Katrin Fischer 2023-04-09 11:36:05 UTC
I am not so happy about the placement. Alternative suggestion: could we place it on the reports start page below the "Other" section?
Comment 5 Katrin Fischer 2023-04-09 11:45:49 UTC
Created attachment 149355 [details] [review]
Bug 33356: [Alternate) Add mana-kb hint to reports start page

An alternate placement of the Mana KB hint on the reports start page.

To test:
* Go to Reports module
* Verify the hint shows when you haven't decided on Mana KB
* Once you set yes or no, it will no longer show
Comment 6 Katrin Fischer 2023-04-09 11:46:29 UTC
What do you think?
Comment 7 David Nind 2023-04-10 19:25:16 UTC
(In reply to Katrin Fischer from comment #6)
> What do you think?

I like your alternative Katrin!

I have tested both patches and they work.

Once it is decided, I am happy to sign off.
Comment 8 Martin Renvoize (ashimema) 2023-04-20 13:26:47 UTC
Katrins is fine.. go with it :)
Comment 9 David Nind 2023-04-20 18:45:34 UTC
Created attachment 149985 [details] [review]
Bug 33356: Add mana-kb hint to reports start page

An alternate placement of the Mana KB hint on the reports start page.

To test:
* Go to Reports module
* Verify the hint shows when you haven't decided on Mana KB
* Once you set yes or no, it will no longer show

Signed-off-by: David Nind <david@davidnind.com>
Comment 10 David Nind 2023-04-20 18:46:38 UTC Comment hidden (obsolete)
Comment 11 David Nind 2023-04-20 18:47:42 UTC Comment hidden (obsolete)
Comment 12 Nick Clemens (kidclamp) 2023-04-28 18:39:09 UTC
Created attachment 150412 [details] [review]
Bug 33356: Add mana-kb hint to reports start page

An alternate placement of the Mana KB hint on the reports start page.

To test:
* Go to Reports module
* Verify the hint shows when you haven't decided on Mana KB
* Once you set yes or no, it will no longer show

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 13 Tomás Cohen Arazi (tcohen) 2023-05-09 13:51:42 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 14 Matt Blenkinsop 2023-05-15 10:14:14 UTC
Enhancement - not backporting to 22.11.x