Bug 13176 - Add links "My account" and "My checkouts" for logged in user to drop down in staff client header
Summary: Add links "My account" and "My checkouts" for logged in user to drop down in ...
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Staff interface (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Marc Véron
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-31 06:12 UTC by Marc Véron
Modified: 2015-06-04 23:33 UTC (History)
3 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 13176 - Add links "My account" and "My checkouts" for logged in user to drop down in staff client header (2.20 KB, patch)
2014-10-31 06:24 UTC, Marc Véron
Details | Diff | Splinter Review
Bug 13176 - Add links "My account" and "My checkouts" for logged in user to drop down in staff client header (2.56 KB, patch)
2014-10-31 06:35 UTC, Marc Véron
Details | Diff | Splinter Review
Bug 13176 - Add links "My account" and "My checkouts" for logged in user to drop down in staff client header (2.26 KB, patch)
2014-10-31 06:46 UTC, Marc Véron
Details | Diff | Splinter Review
Bug 13176 - Add links "My account" and "My checkouts" for logged in user to drop down in staff client header (2.40 KB, patch)
2014-10-31 07:01 UTC, Magnus Enger
Details | Diff | Splinter Review
Bug 13176 - Add links "My account" and "My checkouts" for logged in user to drop down in staff client header (2.67 KB, patch)
2014-10-31 17:43 UTC, Marc Véron
Details | Diff | Splinter Review
[PASSED QA] Bug 13176 - Add links "My account" and "My checkouts" for logged in user to drop down in staff client header (2.75 KB, patch)
2014-10-31 18:01 UTC, Kyle M Hall
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Marc Véron 2014-10-31 06:12:15 UTC
Add direct links to account details and checkout page for logged in user.
Comment 1 Marc Véron 2014-10-31 06:24:07 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2014-10-31 06:35:29 UTC Comment hidden (obsolete)
Comment 3 Marc Véron 2014-10-31 06:46:56 UTC Comment hidden (obsolete)
Comment 4 Magnus Enger 2014-10-31 07:01:55 UTC Comment hidden (obsolete)
Comment 5 Kyle M Hall 2014-10-31 16:57:14 UTC
I can imagine at least some libraries won't want this feature. Can you please add ids and classes to the li tags so they can be easily hidden via css?
Comment 6 Kyle M Hall 2014-10-31 17:09:14 UTC
Also, is it completely necessary to have links to both moremember.pl and circulation.pl? Eventually we may start running out of real estate on our menu bar ; )
Comment 7 Marc Véron 2014-10-31 17:43:37 UTC Comment hidden (obsolete)
Comment 8 Marc Véron 2014-10-31 17:49:28 UTC
Sorry, my comment on the patch was mutilated, it should read:

-----
Added classes "toplinks-myaccount" and "toplink-mycheckouts" to li tags to make it possible to hide them (per Kyle's request).

Switching back to "Signd-off" (Hope this is OK becuause it is a tiny string addition)
-----

PS: Showed it today to a possible customer, they like it.
Comment 9 Kyle M Hall 2014-10-31 18:01:05 UTC
Created attachment 33093 [details] [review]
[PASSED QA] Bug 13176 - Add links "My account" and "My checkouts" for logged in user to drop down in staff client header

This patch adds links to "My account" and "My checkouts" to drop down in staff client header.

To test:
Apply patch
Got to drop down of logged in user (top right)
See new links to "My account" and "My checkout" (above "Log out")
Test the links.

Signed-off-by: Magnus Enger <digitalutvikling@gmail.com>
Works as advertised. The options are not displayed when you are logged
in as the db/admin user.

Added classes "toplinks-myaccount" and "toplink-mycheckouts" to li tags to make it possible to hide them (per Kyle M $
Switching back to "Signd-off" (Hope this is OK becuause it is a tiny string addition)

Marc

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 10 Tomás Cohen Arazi 2014-11-11 12:57:31 UTC
Patch pushed to master.

Nice one, thanks Marc!