carousels: clarify order of records

Bug #1866406 reported by Galen Charlton
42
This bug affects 8 people
Affects Status Importance Assigned to Milestone
Evergreen
Status tracked in Master
3.4
Undecided
Unassigned
3.5
Undecided
Unassigned
3.6
Undecided
Unassigned
Master
Wishlist
Unassigned

Bug Description

It would be useful to have the contents of a carousel have a definite and configurable ordering for display purposes. This would enable several use cases:

- new item carousels that display the newest items first
- manually-organized carousels where the position of a title in the carousel's bucket matched the position that the title would display in the carousel

Evergreen 3.4+

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

+1

tags: added: carousel
Changed in evergreen:
status: New → Confirmed
Revision history for this message
Lindsay Stratton (lstratton) wrote :

I think it would be useful to have an option to set a specific order for records - such as Galen specified, or even for things like series in series order - but ALSO to have an option for dynamic changing order.

I have a carousel that has 60+ titles, I would love if the order were not static so that different titles appear in the first few slots on opening the opac home.

Revision history for this message
Mike Rylander (mrylander) wrote :

I've added a commit to make use of the already-calculated ordering data at the display level. I did not address the "random order" request, as that's definitely a new feature, but I think the basic predictable ordering issue is 100% bug fix.

Branch:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/miker/lp-1866406-carousel-item-order

And, from the commit message:

Previous to this commit, the item display order for carousels was undefined. Now, ordering is stable and predictable.

For Newly Cataloged Item and Newest Items by Shelving Location carousels, the order is from most recently cataloged to least recently cataloged.

For Recently Returned Item carousels, the order is from most recently returned to least recently returned.

For Top Circulated Items carousels, the order is from most circulated to least circulated.

For Manual carousels (as of now, without the ability to adjust the Position of items) it is the order they are added to the backing bucket. Thus emptying and refilling the bucket allows reordering.

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

Since Mike's patch addresses part of the original bug I've split off the other two pieces of this report. See bug 1911938 for manual reordering and bug 1911940 for random ordering.

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

I tested all four types of carousels and it works great! (And I did +1 to the option to randomize ordering on bug 1911940.) My sign off is at:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mccanna/lp1866406-carousel-item-order-signoff

tags: added: signedoff
Revision history for this message
Jason Boyer (jboyer) wrote :

Pushed to master, rel_3_6 and rel_3_5. Thanks Mike and Terran!

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers