Closed
Bug 1270807
Opened 9 years ago
Closed 9 years ago
certerror.whatShouldIDo.badStsCertExplanation lacks a verb
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
FIXED
Firefox 49
Tracking | Status | |
---|---|---|
firefox49 | --- | fixed |
People
(Reporter: Gijs, Unassigned)
References
Details
Attachments
(1 file)
See bug 1240594 comment 43. This was the case prior to bug 1240594, but we should just fix the English string. (AIUI, without rev'ing the id, because it's not changing inasmuch as we're just fixing the English. :flod, is that right?)
Flags: needinfo?(francesco.lodolo)
Reporter | ||
Comment 1•9 years ago
|
||
String for reference:
"This site uses HTTP Strict Transport Security (HSTS) to specify that &brandShortName; only connect to it securely."
Needs "can" or "may" before "only".
Comment 2•9 years ago
|
||
Yes, I think we should update English without changing the ID, the meaning was clear enough. Just in case we can send out an email to double check.
Flags: needinfo?(francesco.lodolo)
Reporter | ||
Comment 3•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/51411/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/51411/
Attachment #8750430 -
Flags: review?(francesco.lodolo)
Comment 4•9 years ago
|
||
Comment on attachment 8750430 [details]
MozReview Request: Bug 1270807 - fix grammar mistake in English version of badStsCertExplanation, r?flod
https://reviewboard.mozilla.org/r/51411/#review48175
Looks good, thanks.
Attachment #8750430 -
Flags: review?(francesco.lodolo) → review+
Comment 6•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
You need to log in
before you can comment on or make changes to this bug.
Description
•