Docs: Carousel documentation clarification

Bug #1901895 reported by Terran McCanna
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Undecided
Unassigned
3.8
Fix Released
Undecided
Unassigned

Bug Description

New bug created from previous omnibus bug: https://bugs.launchpad.net/evergreen/+bug/1851879

1) It would be helpful if the "Age Limit" field description gave examples of how ages should be entered (1 year, 6 months, 10 days, etc.)

2) In the section on the carousel types - in my experience, the "Newly Cataloged" carousel type requires shelving locations to work, but the "Newest Items by Shelving Location" carousel type does not. (This is the opposite of what it says in the documentation.)

3) It would also be helpful if the other carousel type descriptions indicated whether or not shelving locations are required.

Changed in evergreen:
status: New → Confirmed
tags: added: opac-carousel
removed: carousel
Revision history for this message
Gina Monti (gmonti90) wrote :

I created a pull request and this needs a sign off/second look (only because I'm unsure if the corrections I created make any sense):

https://github.com/evergreen-library-system/Evergreen/pull/165

If there's a typo or anything, let me know, though I will read through and check again before committing.

tags: added: pullrequest
Revision history for this message
Gina Monti (gmonti90) wrote :

The pull request is ready for review again. I edited a bit:

https://github.com/evergreen-library-system/Evergreen/pull/165

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

Jane merged this for 3.8+ - marking Fix Committed for 3.8 and 3.9.

Changed in evergreen:
milestone: none → 3.9.1
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.