Typo in Library Setting Description for "How to set default owning library for auto-created line item items"

Bug #2028095 reported by Jennifer Pringle
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Low
Unassigned
3.11
Fix Released
Low
Unassigned

Bug Description

Evergreen 3.11.0

The description for the library setting "How to set default owning library for auto-created line item items" is missing the R in strategy.

Current description:
Stategy to use to set default owning library to set when line item items are auto-created because the provider's default copy count has been set. Valid values are "workstation" to use the workstation library, "blank" to leave it blank, and "use_setting" to use the "Default owning library for auto-created line item items" setting. If not set, the workstation library will be used.

As well, the first sentence of the description is a little hard to follow. Maybe instead the description could be:

Strategy to use when setting the default owning library for line item items that are auto-created due to the the provider's default copy count being set. Valid values are "workstation" to use the workstation library, "blank" to leave it blank, and "use_setting" to use the "Default owning library for auto-created line item items" setting. If not set, the workstation library will be used.

description: updated
tags: added: bitesize
tags: added: acq
Changed in evergreen:
status: New → Incomplete
status: Incomplete → Confirmed
Revision history for this message
Steven Mayo (stmayo) wrote :
tags: added: pullrequest
Revision history for this message
Susan Morrison (smorrison425) wrote :

The original typo was fixed, but there was a new typo introduced in the first sentence of the updated description: "auto-created due to the the provider's default". Once the duplicate 'the' is removed, should be all set.

tags: added: needswork
removed: pullrequest
Revision history for this message
Steven Mayo (stmayo) wrote :
tags: added: pullrequest
removed: needswork
Revision history for this message
Christine Morgan (cmorgan-z) wrote :

Looks good to me!

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

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

Looks good! I realized I can't give the official signoff since I'm from the same org as Steven, but this is good to go.

Galen Charlton (gmc)
tags: added: signedoff
Changed in evergreen:
assignee: nobody → Galen Charlton (gmc)
Revision history for this message
Galen Charlton (gmc) wrote (last edit ):

Pushed down to rel_3_11, along with a follow-up to not update the description if the library changed it first. I also added Susan's implied signoff; multiples don't hurt.

Thanks, Steven, Christine, and Susan!

Changed in evergreen:
importance: Undecided → Low
milestone: none → 3.12.3
status: Confirmed → Fix Committed
assignee: Galen Charlton (gmc) → nobody
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.