Acq: Move Acqusitions Admin Menu

Bug #1361900 reported by Jennifer Pringle
24
This bug affects 5 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned

Bug Description

Wishlist

Evergreen 2.6

To access the admin parts of acquisitions you currently have to go Admin -> Server Administration -> Acquisitions -> Funds (or your desired option). The menus can get unwieldy as they expand out. I also don't think Server Administration is necessarily the best place for acquisitions as some of the admin functions are used on a regular basis by local staff (ie. Funds).

Ideally I'd like to see Admin -> Acquisitions Administration -> Funds (etc.) I think this would make finding the admin side of acquisitions much more intuitive.

tags: added: acq
Kathy Lussier (klussier)
tags: added: bitesize
Changed in evergreen:
assignee: nobody → Michael Peters (mrpeters)
Revision history for this message
Michael Peters (mrpeters) wrote :

Fixed

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mrpeters/lp1361900_move_acquisitions_admin_menu

Note -- this requires that you do 1 of 2 things:

1. Build and distribute a new staff client
2. Replace C:\Program Files (x86)\Evergreen Staff Client Master\chrome\content\main\menu_frame_menus.xul on each installed staff client (assuming you are running master) with the one from this working branch.

A quick look at the commit diff (http://git.evergreen-ils.org/?p=working/Evergreen.git;a=commitdiff;h=feb4c1920610eaec4b827ad80eb73fc86e4e6668) will give you an idea of how you can edit this for other versions on your local machine.

tags: added: pullrequest
Revision history for this message
Ben Shum (bshum) wrote :

Interesting... marking as wishlist and noting that we may want to add a brief release note here as this change will be interesting for new systems. Potential 2.9 alpha target selected if this makes it towards 2.9 changes in general.

Changed in evergreen:
status: New → Confirmed
importance: Undecided → Wishlist
milestone: none → 2.9-alpha
tags: added: needsreleasenote
Revision history for this message
Jennifer Pringle (jpringle-u) wrote :

I have tested this code and consent to signing off on it with my name, Jennifer Pringle, and my email address, <email address hidden>

(We're running this live on 2.8)

tags: added: signedoff
Changed in evergreen:
milestone: 2.9-alpha → 2.9-beta
Revision history for this message
Ben Shum (bshum) wrote :

Made a small change to the XUL file so that we used spaces instead of tabs for the moved menu area.

Pushed to master, along with a brief release note on the change.

Changed in evergreen:
status: Confirmed → Fix Committed
Changed in evergreen:
assignee: Michael Peters (mrpeters) → nobody
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.