"Alias" in Patron Search needs to be labeled "Holds Alias" for clarity/consistency

Bug #1255172 reported by Chris Sharp
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Low
Unassigned

Bug Description

From one of our 2.5 testers comes a request to re-label "Alias" in the Patron Search interface to "Holds Alias". Since 11/2008, the patron registration interface (via the IDL) has labled this "OPAC/Staff Client Holds Alias". I propose changing the Patron Search interface to the abbreviated "Holds Alias" form.

Branch at working:user/csharp/change_alias_label_in_patron_search.

Evergreen Master
OpenSRF 2.2
PostgreSQL 9.1.10
Ubuntu 12.04 LTS

Tags: pullrequest
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.5.1 → 2.5.2
status: New → Confirmed
assignee: Chris Sharp (chrissharp123) → nobody
Dan Wells (dbw2)
Changed in evergreen:
milestone: 2.5.2 → 2.6.0-alpha1
Dan Wells (dbw2)
Changed in evergreen:
milestone: 2.6.0-alpha1 → 2.6.0-beta1
Revision history for this message
Ben Shum (bshum) wrote :

Simple change that I can get behind. Question though, do we backport string changes like this? Or perhaps this is a situation where from 2.6 onwards, we call it "hold alias" but so that translation doesn't get broken for 2.5 and earlier, we shouldn't backport this modification to the language?

Ready to pick and merge this wherever once we know how we would like to proceed.

Changed in evergreen:
assignee: nobody → Ben Shum (bshum)
status: Confirmed → In Progress
Revision history for this message
Dan Wells (dbw2) wrote :

I'm fine with not backporting this for the reason given. Removing the 2.5 target. If others feel differently, we'll need to solve the translation problem before pushing this in (which we should do anyway, at some point, but that would need a champion to step up).

no longer affects: evergreen/2.5
Revision history for this message
Ben Shum (bshum) wrote :

Sounds good. Pushed to master!

Changed in evergreen:
status: In Progress → Fix Committed
assignee: Ben Shum (bshum) → 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.