There's a spurious "to" in the mail/ version of smtpAuthChangeEncryptToPlainSsl

RESOLVED FIXED in Thunderbird 39.0

Status

MailNews Core
Composition
--
trivial
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: nONoNonO, Assigned: Magnus Melin)

Tracking

({regression})

Thunderbird 39.0
regression

Thunderbird Tracking Flags

(thunderbird38+ fixed)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
One of the strings modified in bug 952493 is wrong I think...
It says "try changing to the 'Authentication method'" instead of "try changing the 'Authentication method'" in smtpAuthChangeEncryptToPlainSsl.
(Assignee)

Updated

3 years ago
Keywords: regression

Comment 1

3 years ago
Let's track this in case we decide to do late strings.
tracking-thunderbird38: --- → +
(Assignee)

Updated

3 years ago
Summary: There's a wrong string in the new error messages for composing a mail → There's a spurious "to" in the mail/ version of smtpAuthChangeEncryptToPlainSsl
(Assignee)

Comment 2

3 years ago
Created attachment 8570627 [details] [diff] [review]
bug1137662_smtpAuthChangeEncryptToPlainSsl_typo.patch

I don't think we need to bump the key for this - it's not a "real" semantic change. Let's just land it soon so it doesn't confuse localizers unnecessarily.
Assignee: nobody → mkmelin+mozilla
Status: NEW → ASSIGNED
Attachment #8570627 - Flags: review?(syshagarwal)
Comment on attachment 8570627 [details] [diff] [review]
bug1137662_smtpAuthChangeEncryptToPlainSsl_typo.patch

Review of attachment 8570627 [details] [diff] [review]:
-----------------------------------------------------------------

Sorry for this mistake.

::: mail/locales/en-US/chrome/messenger/messengercompose/composeMsgs.properties
@@ +94,1 @@
>  

Looks good now.
Attachment #8570627 - Flags: review?(syshagarwal) → review+
(Assignee)

Comment 4

3 years ago
https://hg.mozilla.org/comm-central/rev/97ae639ce53d
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 39.0
(Assignee)

Updated

3 years ago
Attachment #8570627 - Flags: approval-comm-aurora+
(Assignee)

Comment 5

3 years ago
https://hg.mozilla.org/releases/comm-aurora/rev/cb0b0c22b3ed
status-thunderbird38: --- → fixed

Comment 6

3 years ago
Is it worth mentioning in m.d.l10n, just in case?
Flags: needinfo?(mkmelin+mozilla)
(Assignee)

Comment 7

3 years ago
Tried it, but posts don't seem to go through. Is mozilla.dev.l10n moderated now?
(Assignee)

Comment 8

3 years ago
Post went through with tb31.
Flags: needinfo?(mkmelin+mozilla)
You need to log in before you can comment on or make changes to this bug.