Circulation Limit Set interface missing sorting, filters, and back button

Bug #1945385 reported by Jennifer Pringle
20
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
3.8
Fix Released
Medium
Unassigned

Bug Description

Evergreen 3.7.0 and 3.7+ (tested on a bug squashing server ahead of 3.8)

The angularized Circulation Limit Set interface does not currently have filters or the ability to sort by any columns. (Administration -> Local Administration -> Circulation Limit Sets)

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

Confirmed still an issue in 3.8.0

Revision history for this message
Michele Morgan (mmorgan) wrote :

Confirmed in 3.7.2

tags: added: admin-pages
Changed in evergreen:
status: New → Confirmed
Revision history for this message
Jennifer Pringle (jpringle-u) wrote :

For anyone who has lots of limit sets like us a workaround is to go back to the dojo interface anytime you need to edit the limits which you can do by going to YOURHOSTNAME/eg/staff/admin/local/config/circ_limit_set

Erica Rohlfs (erohlfs)
Changed in evergreen:
importance: Undecided → Medium
Revision history for this message
Erica Rohlfs (erohlfs) wrote :

Noting that the lack of filter / sorting is particularly problematic with the introduction of the new "Edit Linked Entities" tab. The "linked entities" cannot be created at the same time as the initial limit set creation (the options to add the circ mods, locations, groups are not available during the initial New Circulation Limit Set creation). This means that the user must create the limit set (name, max items, owning library), save, and then the user must search through the list of limit sets to edit the limit set and continue building it with the "linked entities."

Revision history for this message
Erica Rohlfs (erohlfs) wrote :

Also noting that, if the user deletes a limit set, you'll be taken back to the start of the limit set listing. If you edit an existing limit set, clicking Save does not return you back to the limit sets. If you use the Back button, EG takes you back to the start of the limit set listing.

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

Confirmed this is still an issue in 3.9

Changed in evergreen:
assignee: nobody → Jeff Davis (jdavis-sitka)
Revision history for this message
Jeff Davis (jdavis-sitka) wrote :

Working branch user/jeffdavis/lp1945385-sort-and-filter-circ-limit-sets adds sorting and filtering to the circ limit sets UI by refactoring it use eg-admin-page (following the example of bug 1843970):

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jeffdavis/lp1945385-sort-and-filter-circ-limit-sets

Note there's a problem with Linked Shelving Locations which is not addressed by this branch - see bug 1956358.

Changed in evergreen:
assignee: Jeff Davis (jdavis-sitka) → nobody
tags: added: pullrequest
Changed in evergreen:
milestone: none → 3.9.1
Revision history for this message
Jennifer Pringle (jpringle-u) wrote :

I've tested this fix on Jeff's server and it works well for me. I am not signing off since it should be tested by someone from another organization first.

Changed in evergreen:
milestone: 3.9.1 → none
Changed in evergreen:
milestone: none → 3.9.1
Revision history for this message
Jennifer Pringle (jpringle-u) wrote :

Signing off as per discussion in irc

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

tags: added: signedoff
Revision history for this message
Jason Stephenson (jstephenson) wrote :

I tested the fix on Evergreen 3.7.3 and 3.8/master with production data, and it works for me.

I pushed it with Jennifer's signoff added to master, rel_3_9, rel_3_8, and rel_3_7.

Thanks, Jeff, Jennifer, Erica, and Michele!

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

Despite my "testing" this fix on 3.7.3, it doesn't actually work or apply to 3.7. Because of the way that I built the code, I missed the error that occurs when building the Angular staff client (Open-ILS/src/eg2) with the --prod option.

I have set this to "Won't Fix" for 3.7 and removed the 3.7.4 milestone.

Changed in evergreen:
status: Fix Committed → Fix Released
no longer affects: evergreen/3.7
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.