Based on a comment from bshum in IRC this morning, I'm going to target to 2.next for now, but I'm hoping the 2.9 and 2.10 release managers would consider backporting this as a bug fix.
I know there are issues with making string changes between releases, but, at the same time, this is clearly a grammatical error, and, in its current form, it isn't really clear what the string is saying. If I were a patron, I would likely interpret the string as saying that there are 5 holds on this copy, when it really is identifying the queue position. That's giving end users the wrong information.
If the release managers determine it needs to be a new feature, I can write up an associated release notes entry.
Working branch available at http:// git.evergreen- ils.org/ ?p=working/ Evergreen. git;a=shortlog; h=refs/ heads/user/ kmlussier/ lp1554714- hold-count- grammar- fix
Based on a comment from bshum in IRC this morning, I'm going to target to 2.next for now, but I'm hoping the 2.9 and 2.10 release managers would consider backporting this as a bug fix.
I know there are issues with making string changes between releases, but, at the same time, this is clearly a grammatical error, and, in its current form, it isn't really clear what the string is saying. If I were a patron, I would likely interpret the string as saying that there are 5 holds on this copy, when it really is identifying the queue position. That's giving end users the wrong information.
If the release managers determine it needs to be a new feature, I can write up an associated release notes entry.