Evergreen

Angular Staff Catalogue: More Link Missing from Facets

Bug #1934018 reported by Jennifer Pringle
144
This bug affects 30 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Committed
High
Unassigned
Evergreen 3.12-beta
3.11
Fix Committed
Undecided
Unassigned
Evergreen 3.11.2

Bug Description

Evergreen 3.7

The facets in the angular staff catalogue are missing the "More" link which allows staff to see more than the default 5.

I'm not sure if this is related to https://bugs.launchpad.net/evergreen/+bug/1932350

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

Confirmed in a post 3.7.1 master system.

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

Confirmed this is still an issue in 3.9

Michele Morgan (mmorgan)
tags: added: staffcatalogblocker
Terran McCanna (tmccanna)
Changed in evergreen:
importance: Undecided → High
Revision history for this message
Jennifer Weston (jweston) wrote :

(Discussed in Cataloging IG meeting on Oct 11, 2022) -- Workaround is to continue to use the Traditional Search

Revision history for this message
Galen Charlton (gmc) wrote :

Noting for anybody who tackles this that the facets component already has access to the complete list of facet values returned by the search. Consequently, adding a control to display all values for a given facet can be done entirely client-side.

Revision history for this message
Elizabeth Thomsen (et-8) wrote :

Confirmed that this is still an issue in Release 3.11

Revision history for this message
Stephanie Leary (stephanieleary) wrote :

Galen's comment at #5 was very helpful! Here's a branch that adds "More"/"Less" links to expand the facet lists:
https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/sleary/lp1934018-facet-more-link

Screenshot attached.

tags: added: pullrequest
Changed in evergreen:
milestone: none → 3.12-beta
Revision history for this message
Stephanie Leary (stephanieleary) wrote :

Note to anyone setting up test servers for Feedback Fest: I've revised this to be simpler and more efficient, so grab the rebased commit I pushed just now!

Revision history for this message
Susan Morrison (smorrison425) wrote :

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

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

Looks good, thanks Stephanie & Susan!

Fix committed to main and pushed back to 3.11 (did not commit cleanly to 3.10).

Changed in evergreen:
assignee: nobody → Terran McCanna (tmccanna)
status: Confirmed → Fix Committed
Terran McCanna (tmccanna)
Changed in evergreen:
assignee: Terran McCanna (tmccanna) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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