132826 - Need to remove some l10n notes in messenger.properties

Status: VERIFIED FIXED

(MailNews Core :: Localization, defect)

Description

[ji]

Since bug 57440 is fixed, default folder names are localizable now. We need to
remove the following l10n notes in messenger.properties.
# LOCALIZATION NOTES (inboxFolderName): DO NOT TRANSLATE (bugzilla #64199)
inboxFolderName=Inbox
# LOCALIZATION NOTES (trashFolderName): DO NOT TRANSLATE (bugzilla #64199)
trashFolderName=Trash
# LOCALIZATION NOTES (sentFolderName): DO NOT TRANSLATE (bugzilla #64199)
sentFolderName=Sent
# LOCALIZATION NOTES (draftsFolderName): DO NOT TRANSLATE (bugzilla #64199)
draftsFolderName=Drafts
# LOCALIZATION NOTES (templatesFolderName): DO NOT TRANSLATE (bugzilla #23625).
# Fix to bug 23625 requires to create template folder URI for new IMAP account, 
# which requires the folder name to be picked from here like for other special
folders
templatesFolderName=Templates
# LOCALIZATION NOTES (unsentFolderName): DO NOT TRANSLATE (bugzilla #64199)
unsentFolderName=Unsent Messages

Comment 1

[nhottanscp]

L10N issue.
L10N group, please triage.

Comment 2

[rchen]

Will fix it.

->nsbeta1+.

Comment 3

[rchen]

Attachment: patch for L10N notes update. It doesn't impact application but giving the correct instruction.

Comment 4

[rchen]

Ying, Please info HDT to localize the folder names.

Comment 5

[nhottanscp]

Comment on attachment 76773 [details] [diff] [review]
patch for L10N notes update. It doesn't impact application but giving the correct instruction.

r=nhotta

Ray, do you want to put a comment for the remaining issue (bug 129530)?

Comment 6

[rchen]

If they are the same strings and related, I can. Can you provide the patch? 

Comment 7

[Frank Tang]

132826 Need to remove some l10n notes in messenger.properties
Impact Summery
Impact Platform: Localization Tools
Impact language users: ALL localize build
Probability of hitting the problem: N/A
Severity if hit the problem in the worst case: N/A
Way of recover after hit the problem: N/A
Risk of the fix: NONE
Potential benefit of fix this problem:

This is not a BUG bug a task log. It require to change the comment part of
localizable file so the localization will begin to translate some strings that
we don't want them to translate. It is NO risk fix.

Comment 8

[Frank Tang]

This should be adt2 since it will impact all the localization projects based on
the English dtd/property files. Without this change, those ~50 localization
won't pick up the changes. 

Comment 9

[Michele Carlson]

Ray is out on vacation this week. If this needs to get fixed this week, perhaps
we should reassign it? Frank - Who do you suggest?

Comment 10

[Frank Tang]

reassign to yokoyama
roy- can you ask for superreview ?

this is a NO RISK high impact bug (impact all localization , German, Franch,
Japanese, Spanish, etc).

roy- after got sr, please mark as adt1.0.0 in the keywords field. thanks

Comment 11

[Frank Tang]

roy- please help. 

Comment 12

[Roy Yokoyama]

only messenger.properties file change.
No need for /sr.  Ask for /a

Comment 13

[Randell Jesup [:jesup] (needinfo me)]

Comment on attachment 76773 [details] [diff] [review]
patch for L10N notes update. It doesn't impact application but giving the correct instruction.

a=[e-mail] under the assumption that modifications to this file do not
require sr=.  If they do, please obtain sr= before checkin

Comment 14

[Roy Yokoyama]

Please give adt1.0.0+

Comment 15

[Jaime Rodriguez, Jr.]

adt1.0.0+ (on ADT's behalf) for approval to checkin to 1.0. Pls check this in
today. Low risk, that only impacts L10N process.

Comment 16

[Roy Yokoyama]

checked-in

Comment 17

[Roy Yokoyama]

marking as FIXED

Comment 18

[ji]

Verified as fixed.