Web Client: Lost Circ History by Year Info

Bug #1743611 reported by Terran McCanna
158
This bug affects 33 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
High
Unassigned
3.6
Fix Released
High
Unassigned

Bug Description

In 3.0.2:

When using the Item Status screen in 3.0, we are no longer able to hover over the Total Circs number and get a breakdown of circs by year that we had available in the xul client. This information is extremely helpful when weeding.

Revision history for this message
Jennifer Pringle (jpringle-u) wrote :

Confirmed this information is still missing in 3.1.

Changed in evergreen:
status: New → Confirmed
Changed in evergreen:
importance: Undecided → Medium
Revision history for this message
Jeff Godin (jgodin) wrote :

In addition to the weeding scenario Terran mentioned above, this information is also useful when making ad-hoc decisions to reorder an item that has been declared lost/damaged.

It appears that the underlying API call open-ils.pcrud.search.circbyyr.atomic contains the data needed, and we're already making that call in the UI in question.

We just need to display the data.

uib-tooltip or perhaps even uib-popover seem like good possibilities here.

Changed in evergreen:
assignee: nobody → Jeff Godin (jgodin)
Revision history for this message
Jane Sandberg (sandbergja) wrote :

Thanks for taking this on, Jeff. It would be great if your version is more noticeable than the XUL version was. Maybe a little icon next to the input that provides the tooltip or popover on focus?

Revision history for this message
Mary Llewellyn (mllewell) wrote :

I have pictures in case they'd be useful.

Revision history for this message
Terran McCanna (tmccanna) wrote :

Bumped to High since it has so much heat.

Changed in evergreen:
importance: Medium → High
Revision history for this message
Jason Stephenson (jstephenson) wrote :

Jeff, are you still/currently working on this?

Revision history for this message
Jeff Godin (jgodin) wrote :

Jason- Not working on this today, no. Unassigned myself, but may still work on it later.

Changed in evergreen:
assignee: Jeff Godin (jgodin) → nobody
tags: added: regression
tags: removed: webstaffclient
tags: added: circulation
Revision history for this message
John Amundson (jamundson) wrote :

We continue to get requests for this feature to come back. For some of our libraries, this is the "one" thing that is missing in the web client. We would love to see some movement on this.

Revision history for this message
Jennifer Bruch (jbruchpails) wrote :

Reports are not able to make up for the loss of this feature. Would really like to see it back or if necessary a way to replicate it in reports.

Dan Briem (dbriem)
Changed in evergreen:
assignee: nobody → Dan Briem (dbriem)
Revision history for this message
Dan Briem (dbriem) wrote :

Branch for testing: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dbriem/lp1743611_circ_history_by_year

Adds an info button next to Total Circs in the AngularJS staff client Item Status screen. Clicking the button displays a popover list of total circs by year.

I went with a click action instead of hovering for usability reasons, for example staff may want to copy and paste the popover info. The button is also tab accessible and formatted the same as the other icon buttons in the Item Status interface.

tags: added: pullrequest
Changed in evergreen:
assignee: Dan Briem (dbriem) → nobody
Revision history for this message
Michele Morgan (mmorgan) wrote :

This works well! If the item has no circ history, the info button next to Total Circs does not appear, which is appropriate.

I'm attaching a screenshot of what this looks like.

My signoff is at:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mmorgan/lp1743611_signoff

user/mmorgan/lp1743611_signoff

tags: added: signedoff
Changed in evergreen:
milestone: none → 3.7.2
Changed in evergreen:
assignee: nobody → Jason Stephenson (jstephenson)
Revision history for this message
Jason Stephenson (jstephenson) wrote :

We have tested this at CW MARS as well, and it works for us. I've added John Amundson's signoff per his instruction, as well as my own, and pushed to mater, rel_3_7, and rel_3_6.

I set this to "Won't Fix" for 3.5 because that branch is out of support and this is not a security fix.

Thanks, everyone!

Changed in evergreen:
status: Confirmed → Fix Committed
assignee: Jason Stephenson (jstephenson) → nobody
Revision history for this message
Michele Morgan (mmorgan) wrote :

A tiny tweak to this is needed. See follow up bug 1942920.

no longer affects: evergreen/3.5
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.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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