Web Client: Holds Pull List columns missing or missing data

Bug #1728147 reported by Michele Morgan
52
This bug affects 8 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
High
Unassigned
3.0
Fix Released
High
Unassigned

Bug Description

The Web Client Holds Pull List has a few column issues:

Missing columns:

- Hold Type
- Edition
- Expire Date
- ISBN
- Staff Hold
- Top of Queue
- Total Holds

Problem Columns:

- "Parts" column does not display data. Ideally it should display the Part data whether the hold is a Copy or Part type hold.

- "Shelving Location" and "Shelving Location Label" are duplicates.

- "Call Number" displays the data for Call Number Prefix twice, followed by the Call Number Label, followed by the data for Call Number Suffix twice. So for an item with prefix "Fiction", label "Grisham", suffix "Summer Reading" the Pull List Call Number field displays as "Fiction Fiction Grisham Summer Reading Summer Reading". Perhaps this one should be removed given that...

- "Call Number Label" displays Prefix, Call Number Label, Suffix, which works beautifully. So maybe this column should be renamed "Call Number"

- "Hold Notes" displays an array of database ids for hold notes.

- "SMS Carrier" is not useful without other notification methods so either it should be removed, or renamed to "Text Carrier", and the other notification columns should be added "Text Notify", "Phone Notify" and "Email Notify"

Setting this as High priority as the absence of Part data means the pull list doesn't work for part level holds.

Kathy Lussier (klussier)
tags: added: webstaffclient webstaffcolumns
Revision history for this message
Scott Thomas (scott-thomas-9) wrote :

Also, none of the Item Columns are available. We make great use of Item: Copy Status Changed Time.

Revision history for this message
Kathy Lussier (klussier) wrote :

I've also heard a request for circulation modifier from the item class. Are there other item fields that are important to display here? There are a lot of item columns in the xul client pull list that seem like they are really unnecessary for this interface (create date, active date, cost, is holdable, is deleted).

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

+1 for the Circulation Modifier

Changed in evergreen:
status: New → Confirmed
Cesar V (cesardv)
Changed in evergreen:
assignee: nobody → Cesar V (cesardv)
Cesar V (cesardv)
Changed in evergreen:
assignee: Cesar V (cesardv) → nobody
Revision history for this message
Mary Llewellyn (mllewell) wrote :

+1 Circulation modifier

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

We've received a request to add patron first and last name as well. (These were available in the xul client.)

Revision history for this message
Andrea Neiman (aneiman) wrote :

PaILS has contracted with Equinox to address this bug. Terran, unfortunately the list of fields I wrote into the contract was as of 2/2/18 so when we post a branch I will open a new wishlist bug for first/last name.

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

Thanks Andrea / PaILS!

Cesar V (cesardv)
Changed in evergreen:
assignee: nobody → Cesar V (cesardv)
Revision history for this message
Cesar V (cesardv) wrote :
Andrea Neiman (aneiman)
tags: added: pullrequest
Changed in evergreen:
milestone: none → 3.1-beta
Cesar V (cesardv)
Changed in evergreen:
assignee: Cesar V (cesardv) → nobody
Galen Charlton (gmc)
no longer affects: evergreen/3.1
Andrea Neiman (aneiman)
Changed in evergreen:
milestone: 3.1-beta → 3.1-rc
Kathy Lussier (klussier)
Changed in evergreen:
assignee: nobody → Kathy Lussier (klussier)
Revision history for this message
Kathy Lussier (klussier) wrote :

This is a big improvement! I found a couple of issues and also have a couple of questions.

* The Staff Hold field isn't populating for staff-placed holds.
* The hold notes are surrounded by brackets, which don't look too bad when there is a hold note, but also displays when there is not a hold note. Can the brackets be removed? See the screenshot at http://www.screencast.com/t/kfibSzTT46.
* I noticed that one of the call number fields was removed, as was suggested by Michele. I haven't tested this yet, but is the remaining call number field one that sorts by the call number sortkey? If I remember correctly, only one of those call number fields sorted by sortkey, and I just want to make sure we didn't lose that one. I can test this on my own over the next couple of days.
* I see the notification fields Michele mentioned were added to the column pickers, but the SMS carrier was also removed. I don't know if it's important to have there.

Thanks Cesar!

Kathy Lussier (klussier)
Changed in evergreen:
assignee: Kathy Lussier (klussier) → nobody
Galen Charlton (gmc)
Changed in evergreen:
assignee: nobody → Galen Charlton (gmc)
Revision history for this message
Galen Charlton (gmc) wrote :

The branch user/gmcharlt/lp1728147_tweaked contains a set of additions that addresses most of the concerns raised by Kathy in comment #9. The one exception is the rendering of the JSON array values for the notes field; this is something that is probably better handled within egGrid itself and would take a bit more time than is available for the 3.1-rc cutoff. However, I think with the new branch, we're closer to something worth putting in the RC.

Changed in evergreen:
assignee: Galen Charlton (gmc) → nobody
Kathy Lussier (klussier)
Changed in evergreen:
assignee: nobody → Kathy Lussier (klussier)
Revision history for this message
Kathy Lussier (klussier) wrote :

Looks great! Thanks Galen and Cesar!

Merged to master and release 3.0.

Changed in evergreen:
status: Confirmed → Fix Committed
assignee: Kathy Lussier (klussier) → nobody
Changed in evergreen:
status: Fix Committed → Fix Released
Revision history for this message
Scott Thomas (scott-thomas-9) wrote :

We are happy this is fixed, but we need the CN Prefix and CN Suffix field separated out like in XUL. Is this possible?

Scott

Revision history for this message
Andrea Neiman (aneiman) wrote :

A patch to restore prefix/suffix as separate columns is on bug 1765731.

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.