Circulation Modifier Should display in Public Catalog Course Details Page

Bug #1935693 reported by Jane Sandberg
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
3.6
Fix Released
Medium
Unassigned

Bug Description

In the public catalog course details page, the circulation modifier displays to the public for items that are associated with a course. This is so patrons can get a sense for the item's loan duration.

The circ modifier should also display on the public catalog's course detail page, for the same reason.

I will be working on this on behalf of BC Libraries Cooperative.

Revision history for this message
Jane Sandberg (sandbergja) wrote :

While working on this, I noticed some strings that were not marked for translation on this screen, so I included those in my pull request.

Here is a branch: user/sandbergja/lp1935693_circ_modifier_on_course_page

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

And here are the testing notes from the commit message:

1) Enable the course materials module
2) Add some courses
3) Add some materials to those courses, using a mix of circulation modifiers
4) In the OPAC, use the Course Search to search for your courses
5) Click on the course you created
6) Note that physical items associated with the course display their circulation modifiers in the circulation modifier column.

Changed in evergreen:
assignee: Jane Sandberg (sandbej) → nobody
tags: added: pullrequest
Revision history for this message
Beth Willis (willis-a) wrote :

This code works as advertised. The circulation modifiers for items that are associated with active courses display to the public on the Course Details page.

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

tags: added: signed-off
Revision history for this message
Jane Sandberg (sandbergja) wrote :

Thanks, Beth! I pushed another commit to address an issue found with this table in testing: long titles have a tendency to become very wide and squish the other columns. The second commit in this branch should prevent this issue.

tags: removed: signed-off
Revision history for this message
Christine Burns (christine-burns) wrote :

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

long titles are wrapped so the other columns are no longer squished.

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

Pushed down to rel_3_6. Thanks, Jane, Beth, and Christine!

Changed in evergreen:
status: New → Confirmed
importance: Undecided → Medium
milestone: none → 3.7.2
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.