action.hold_request_cancel_cause entries not marked for translation

Bug #1160270 reported by Pasi Kallinen
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Undecided
Unassigned
2.3
Fix Released
Undecided
Unassigned
2.4
Fix Released
Undecided
Unassigned

Bug Description

*EG master

action.hold_request_cancel_cause inserts are not marked with oils_i18n_gettext(), nor would they be picked up into po-files if they were, as they're not in 950.data.seed-values.sql

Ben Shum (bshum)
Changed in evergreen:
status: New → Triaged
Revision history for this message
Pasi Kallinen (paxed) wrote :

Fix in user/paxed/lp1160270

tags: added: pullrequest
Changed in evergreen:
milestone: none → 2.5.0-m2
Revision history for this message
Remington Steed (rjs7) wrote :

This bug is a duplicate of:

https://bugs.launchpad.net/evergreen/+bug/1107808

I have marked that as the duplicate, since this bug seems to have the better title. The submitted branches on each are essentially identical.

Revision history for this message
Remington Steed (rjs7) wrote :

Since this is a simple change, I have only done the basic test of running the SQL inserts in question, and they work as expected. Also, it is my understanding that this change does not require an upgrade script, because the upgrade process doesn't yet participate in the translation-building process. If this is true, then perhaps we should open a bug to address this.

I also resolved a merge conflict, for a commit that adds a new cancel cause "Patron via SIP". Signed off and rebased to master. Thanks Pasi!

lp1160270_action_i18n_seed_data

Revision history for this message
Remington Steed (rjs7) wrote :
Revision history for this message
Dan Wells (dbw2) wrote :

Picked back through 2.3. Thanks again, guys!

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

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.