Bug 10415 - Add course reserves to staff client home page
Summary: Add course reserves to staff client home page
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Staff interface (show other bugs)
Version: master
Hardware: All All
: P5 - low normal (vote)
Assignee: Owen Leonard
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-06-05 12:58 UTC by Owen Leonard
Modified: 2015-06-04 23:30 UTC (History)
6 users (show)

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


Attachments
Bug 10415 - Add course reserves to staff client home page (30.35 KB, patch)
2014-03-11 15:26 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 10415 - Add course reserves to staff client home page (30.37 KB, patch)
2014-03-12 15:23 UTC, Biblibre Sandboxes
Details | Diff | Splinter Review
Bug 10415 - Add course reserves to staff client home page (30.43 KB, patch)
2014-03-12 15:58 UTC, Biblibre Sandboxes
Details | Diff | Splinter Review
[PASSED QA] Bug 10415 - Add course reserves to staff client home page (30.66 KB, patch)
2014-03-12 18:10 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 Owen Leonard 2013-06-05 12:58:49 UTC
If course reserves is an important enough module to list in the global "More" menu then it should follow that it is important enough to include on the staff client home page. The menu choices in the header menu (with the exception of "Cart" mirror the main module links on the staff client home page.
Comment 1 Owen Leonard 2013-06-05 13:00:28 UTC
Some possible choices for a "course reserves" icon for the home page:

http://thenounproject.com/noun/stack-of-books/
Comment 2 Nicole C. Engard 2013-07-29 14:55:37 UTC
I vote :

http://thenounproject.com/noun/stack-of-books/#icon-No11494 

or 

http://thenounproject.com/noun/stack-of-books/#icon-No3700

And I agree that we need a module link for it on the home page.

Nicole
Comment 3 Galen Charlton 2014-03-10 17:57:34 UTC
(In reply to Nicole C. Engard from comment #2)
> I vote :
> 
> http://thenounproject.com/noun/stack-of-books/#icon-No11494 

I vote for this one.
Comment 4 Owen Leonard 2014-03-11 15:26:21 UTC Comment hidden (obsolete)
Comment 5 jmbroust 2014-03-12 13:36:26 UTC
Tested on sandbox 15 : I can't see course reserves anywhere on the homepage nor in the header menu. I tried to clear the browser cache but it's the same.That's strange, something is wrong with cache. Someone else to test ?
Comment 6 Owen Leonard 2014-03-12 14:42:19 UTC
Is the "UseCourseReserves" system preference enabled?
Comment 7 jmbroust 2014-03-12 15:04:41 UTC
(In reply to Owen Leonard from comment #6)
> Is the "UseCourseReserves" system preference enabled?

sorry, i missed this step ! i return testing, thanks.
Comment 8 Biblibre Sandboxes 2014-03-12 15:23:05 UTC
Patch tested with a sandbox, by Broust <jean-manuel.broust@univ-lyon2.fr>
Comment 9 Biblibre Sandboxes 2014-03-12 15:23:25 UTC Comment hidden (obsolete)
Comment 10 Biblibre Sandboxes 2014-03-12 15:58:06 UTC
Patch tested with a sandbox, by marjorie barry-vila <marjorie.barry-vila@ccsr.qc.ca>
Comment 11 Biblibre Sandboxes 2014-03-12 15:58:25 UTC Comment hidden (obsolete)
Comment 12 Katrin Fischer 2014-03-12 18:10:29 UTC
Created attachment 26217 [details] [review]
[PASSED QA] Bug 10415 - Add course reserves to staff client home page

A link to course reserves is in the global header menu but not on the
home page. All links from the global header should be present on the
home page as well. This patch adds it.

To test, apply the patch and if necessary clear your browser cache. View
the staff client home page. If you have "UseCourseReserves" enabled you
should see a link for the course reserves page which is visually
consistent with the other module links. If you do not have course
reserves enabled you should not see the link.

Unrelated: I positioned the admin link after the tools link because it
bugged me.

Signed-off-by: Broust <jean-manuel.broust@univ-lyon2.fr>
Signed-off-by: marjorie barry-vila <marjorie.barry-vila@ccsr.qc.ca>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Passes all tests and QA script, works as described.
Course reserves is still accessible without permissions, but
you can't make any changes to the reserves then.
Comment 13 Galen Charlton 2014-03-12 18:20:49 UTC
Pushed to master.  Thanks, Owen!
Comment 14 Fridolin Somers 2014-06-12 07:25:46 UTC
Pushed to 3.14.x, will be in 3.14.08