Add to Carousel Missing from Other Actions on Bib Record in Angular

Bug #1922120 reported by John Yorio
30
This bug affects 6 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
3.6
Fix Released
Medium
Unassigned

Bug Description

3.6

To reproduce the issue:
Go to Cataloging
Click Search the Catalog
Perform a search
View the bib record for one of the results
Click on Other Actions
Notice that Add to Carousel is not present

Now, go to Cataloging
Click Search the Catalog (Traditional)
Perform the same search
View a bib record for one of the results
Click on Other Actions
Notice that Add to Carousel *is* present.

Expected result:
I expected it to be visible for both.

Garry Collum (gcollum)
Changed in evergreen:
status: New → Confirmed
Revision history for this message
Millissa (millissam) wrote :

Our library was using this option in the prior version and really miss having it in 3.6.

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

The PaILS SPARK libraries use this option all the time and we are not looking forward to its loss on our next update.

Changed in evergreen:
assignee: nobody → Jane Sandberg (sandbej)
Changed in evergreen:
assignee: Jane Sandberg (sandbej) → nobody
importance: Undecided → Medium
Revision history for this message
Jane Sandberg (sandbergja) wrote :

Here is a branch that brings this feature over to the Angular catalog: user/sandbergja/LP1922120_ng_add_to_carousel

Here is a link: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/sandbergja/LP1922120_ng_add_to_carousel

And here are the testing notes from the commit message:

1) Create several manual carousels, and make sure they are
set to active.
2) Open a bib record in the Angular staff catalog.
3) Under Other Actions, choose the "Add to Carousel" action
4) Choose your preferred carousel.
5) Click Add to carousel.
6) Go back to carousel administration (or look at the carousel
itself), and confirm that the record has been added to the
carousel in question.

One note for anybody trying to apply this patch to older systems: it relies on the open-ils.actor.container.item.create.batch API introduced in 3.6.3. I chose that API so that the new add-to-carousel-dialog component could theoretically be re-used in screens that allow users to add multiple records to a carousel at once, without sending a ton of OpenSRF calls.

tags: added: pullrequest
Changed in evergreen:
milestone: none → 3.7.1
Changed in evergreen:
assignee: nobody → Terran McCanna (tmccanna)
Revision history for this message
Terran McCanna (tmccanna) wrote :
tags: added: signedoff
Changed in evergreen:
assignee: Terran McCanna (tmccanna) → nobody
Revision history for this message
Galen Charlton (gmc) wrote :

Pushed down to rel_3_6. Thanks, Jane and Terran!

Changed in evergreen:
status: Confirmed → Fix Committed
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.

Other bug subscribers

Remote bug watches

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