Closed
Bug 1046178
Opened 10 years ago
Closed 10 years ago
Prevent using hardcoded brand names in l10n data files
Categories
(Hello (Loop) :: Client, defect, P1)
Hello (Loop)
Client
Tracking
(firefox34 fixed)
Tracking | Status | |
---|---|---|
firefox34 | --- | fixed |
People
(Reporter: mikedeboer, Assigned: alexbardas)
References
Details
Attachments
(1 file, 1 obsolete file)
5.33 KB,
patch
|
alexbardas
:
review+
|
Details | Diff | Splinter Review |
At the moment, brand names are hardcoded in our l10n data files. We'd probably want to use brandShortname wherever it now says 'Firefox' and clientShortname wherever it now says 'Loop'.
Comment 1•10 years ago
|
||
Don't we want clientShortname to be "WebRTC" and later switch to "Hello" ?
[0] https://bugzilla.mozilla.org/show_bug.cgi?id=1000127#c4
Comment 2•10 years ago
|
||
I'm removing "first release needed" because we want this independent of the direct calling and contacts work.
Whiteboard: [qa-][first release needed] → [qa-]
Updated•10 years ago
|
Target Milestone: mozilla34 → 34 Sprint 1- 8/4
Reporter | ||
Updated•10 years ago
|
Status: NEW → ASSIGNED
Updated•10 years ago
|
Flags: firefox-backlog+
Reporter | ||
Comment 3•10 years ago
|
||
Un-assigning, because it's unlikely that I'll be working on this before my PTO starts.
Assignee: mdeboer → nobody
Status: ASSIGNED → NEW
Comment 4•10 years ago
|
||
Setting this as blocking bug 1000269 as that's when we'll want it before.
Blocks: 1000269
Updated•10 years ago
|
Whiteboard: [qa-] → [p=2, qa-]
Updated•10 years ago
|
Iteration: 34.1 → ---
Updated•10 years ago
|
Target Milestone: 34 Sprint 1- 8/4 → mozilla34
Updated•10 years ago
|
Whiteboard: [p=2, qa-] → [qa-]
Updated•10 years ago
|
Flags: qe-verify-
Whiteboard: [qa-]
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → abardas
Status: NEW → ASSIGNED
Updated•10 years ago
|
Iteration: --- → 34.3
Assignee | ||
Comment 5•10 years ago
|
||
I didn't find other files dealing with l10n in loop. I've locally tested in both Firefox and Safari (which doesn't support WebRTC).
Attachment #8476384 -
Flags: review?(adw)
Comment 6•10 years ago
|
||
Comment on attachment 8476384 [details] [diff] [review]
Prevent using hardcoded brand names in l10n data files
This localization note,
> ## LOCALIZATION NOTE (legal_text_and_links): In this item, don't translate the
> ## part between {{..}}
should be moved up since the {{..}} are now found higher up and throughout the document. Maybe just put this above any of the l10n strings.
Attachment #8476384 -
Flags: review?(adw) → review+
Assignee | ||
Comment 7•10 years ago
|
||
Attachment #8476384 -
Attachment is obsolete: true
Attachment #8476469 -
Flags: review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 8•10 years ago
|
||
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 10•10 years ago
|
||
Untracking for QA. Please needinfo me to request specific testing.
Whiteboard: [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•