Add gmail to the localizations that want that

RESOLVED FIXED

Status

Mozilla Localizations
Other
--
major
RESOLVED FIXED
9 years ago
2 years ago

People

(Reporter: Pike, Assigned: Pike)

Tracking

({fixed1.9.0.2, verifyme})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

9 years ago
We want to add Gmail to a bunch of locales, I'd create a patch for that. En-US was bug 444395.

Mic collected a list of locales that want that.

Things to watch out for are at least Gmail vs Googlemail (just de, en-GB?), and I saw some locales asking to drop yahoo mail. Do we want to do that? If so, I'll need to treat those differently.

Mic, can you follow up here with a list of locales? Just language code per procedure would do.

Comment 1

9 years ago
hi
here are the list of locales who have opted in via google group notice or bugmail. There are a few that are still to be confirmed (pt-PT, gu-IN, el). I will verify those asap

af
ar
as
be
bg
bn
ca
cs
da
de
en-gb
es-ar
es-ES
eu
fi
fr
fy-nl
ga-ie
gl
gl
he
hi
hu
id
is
it
ja
ko
kn
lt
mk
nb-no
nn-no
pa
pl
pt-br
ro
ru
si
sk
sl
sq
sv-SE
te
th
th
uk
zh-cn
zh-tw

Comment 2

9 years ago
pt-PT opted in (10-07-2008 15:37 (GMT+1) according to my sent folder)
(Assignee)

Comment 3

9 years ago
Created attachment 334690 [details] [diff] [review]
add Gmail/Googlemail to locales

This patch for reference, and inspection.

This is the list I used for copy'n'paste:

Gmail:
af ar as be bg bn-IN ca cs da es-AR es-ES eu fi fr fy-NL ga-IE gl gl he hi-IN hu id is it ja ja-JP-mac ko kn lt mk nb-NO nn-NO pa-IN pl pt-BR pt-PT ro ru si sk sl sq sv-SE te th th uk zh-CN zh-TW

Googlemail:
de en-GB

As per irc discussion with Mic, we didn't do any yahoo removals.
(Assignee)

Comment 4

9 years ago
Created attachment 334693 [details] [diff] [review]
add Gmail/Googlemail to locales, v2

I adjusted the spacing around the '=' to be more like en-US region.properties, and removed one gl from the list, it was on there twice, leading to a double gmail handler.

Updated list for Gmail:
af ar as be bg bn-IN ca cs da es-AR es-ES eu fi fr fy-NL ga-IE gl he hi-IN hu id is it ja ja-JP-mac ko kn lt mk nb-NO nn-NO pa-IN pl pt-BR pt-PT ro ru si sk sl sq sv-SE te th th uk zh-CN zh-TW
Attachment #334690 - Attachment is obsolete: true
(Assignee)

Comment 5

9 years ago
Comment on attachment 334693 [details] [diff] [review]
add Gmail/Googlemail to locales, v2

Thai is doubled, too.

Not mentioned so far, but in all-locales are et, hy-AM, ka, ku, ml, mn, mr, nl, sr, tr

mr is bug 446613 comment 8, adding that, too.

New patch coming up.
Attachment #334693 - Attachment is obsolete: true
(Assignee)

Comment 6

9 years ago
Created attachment 334695 [details] [diff] [review]
add Gmail/Googlemail to locales, v3

Yet another updated locale list for Gmail:

af ar as be bg bn-IN ca cs da es-AR es-ES eu fi fr fy-NL ga-IE gl he hi-IN hu id is it ja ja-JP-mac ko kn lt mk mr nb-NO nn-NO pa-IN pl pt-BR pt-PT ro ru si sk sl sq sv-SE te th uk zh-CN zh-TW

Googlemail:

de en-GB
(Assignee)

Comment 7

9 years ago
Comment on attachment 334695 [details] [diff] [review]
add Gmail/Googlemail to locales, v3

Phu, finally, a=me for me landing this on cvs trunk for 3.0.2.

I'll probably abuse this patch to generate a series of patches to land on hg to finalize the l10n buildbot work.
Attachment #334695 - Flags: approval1.9.0.2+
(Assignee)

Comment 8

9 years ago
Tim, can QA help out with spot tests to verify this?
Keywords: fixed1.9.0.2, verifyme
(Assignee)

Comment 9

9 years ago
... which means to say that I landed this on CVS trunk.

Checking in af/browser/chrome/browser-region/region.properties;
/l10n/l10n/af/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.4; previous revision: 1.3
done
Checking in ar/browser/chrome/browser-region/region.properties;
/l10n/l10n/ar/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.6; previous revision: 1.5
done
Checking in as/browser/chrome/browser-region/region.properties;
/l10n/l10n/as/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.2; previous revision: 1.1
done
Checking in be/browser/chrome/browser-region/region.properties;
/l10n/l10n/be/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.7; previous revision: 1.6
done
Checking in bg/browser/chrome/browser-region/region.properties;
/l10n/l10n/bg/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.7; previous revision: 1.6
done
Checking in bn-IN/browser/chrome/browser-region/region.properties;
/l10n/l10n/bn-IN/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.5; previous revision: 1.4
done
Checking in ca/browser/chrome/browser-region/region.properties;
/l10n/l10n/ca/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.15; previous revision: 1.14
done
Checking in cs/browser/chrome/browser-region/region.properties;
/l10n/l10n/cs/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.9; previous revision: 1.8
done
Checking in da/browser/chrome/browser-region/region.properties;
/l10n/l10n/da/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.6; previous revision: 1.5
done
Checking in de/browser/chrome/browser-region/region.properties;
/l10n/l10n/de/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.9; previous revision: 1.8
done
Checking in en-GB/browser/chrome/browser-region/region.properties;
/l10n/l10n/en-GB/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.8; previous revision: 1.7
done
Checking in es-AR/browser/chrome/browser-region/region.properties;
/l10n/l10n/es-AR/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.11; previous revision: 1.10
done
Checking in es-ES/browser/chrome/browser-region/region.properties;
/l10n/l10n/es-ES/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.8; previous revision: 1.7
done
Checking in eu/browser/chrome/browser-region/region.properties;
/l10n/l10n/eu/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.13; previous revision: 1.12
done
Checking in fi/browser/chrome/browser-region/region.properties;
/l10n/l10n/fi/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.19; previous revision: 1.18
done
Checking in fr/browser/chrome/browser-region/region.properties;
/l10n/l10n/fr/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.22; previous revision: 1.21
done
Checking in fy-NL/browser/chrome/browser-region/region.properties;
/l10n/l10n/fy-NL/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.10; previous revision: 1.9
done
Checking in ga-IE/browser/chrome/browser-region/region.properties;
/l10n/l10n/ga-IE/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.14; previous revision: 1.13
done
Checking in gl/browser/chrome/browser-region/region.properties;
/l10n/l10n/gl/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.3; previous revision: 1.2
done
Checking in he/browser/chrome/browser-region/region.properties;
/l10n/l10n/he/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.11; previous revision: 1.10
done
Checking in hi-IN/browser/chrome/browser-region/region.properties;
/l10n/l10n/hi-IN/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.6; previous revision: 1.5
done
Checking in hu/browser/chrome/browser-region/region.properties;
/l10n/l10n/hu/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.10; previous revision: 1.9
done
Checking in id/browser/chrome/browser-region/region.properties;
/l10n/l10n/id/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.3; previous revision: 1.2
done
Checking in is/browser/chrome/browser-region/region.properties;
/l10n/l10n/is/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.3; previous revision: 1.2
done
Checking in it/browser/chrome/browser-region/region.properties;
/l10n/l10n/it/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.21; previous revision: 1.20
done
Checking in ja-JP-mac/browser/chrome/browser-region/region.properties;
/l10n/l10n/ja-JP-mac/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.11; previous revision: 1.10
done
Checking in ja/browser/chrome/browser-region/region.properties;
/l10n/l10n/ja/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.11; previous revision: 1.10
done
Checking in kn/browser/chrome/browser-region/region.properties;
/l10n/l10n/kn/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.3; previous revision: 1.2
done
Checking in ko/browser/chrome/browser-region/region.properties;
/l10n/l10n/ko/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.11; previous revision: 1.10
done
Checking in lt/browser/chrome/browser-region/region.properties;
/l10n/l10n/lt/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.9; previous revision: 1.8
done
Checking in mk/browser/chrome/browser-region/region.properties;
/l10n/l10n/mk/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.9; previous revision: 1.8
done
Checking in mr/browser/chrome/browser-region/region.properties;
/l10n/l10n/mr/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.4; previous revision: 1.3
done
Checking in nb-NO/browser/chrome/browser-region/region.properties;
/l10n/l10n/nb-NO/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.10; previous revision: 1.9
done
Checking in nn-NO/browser/chrome/browser-region/region.properties;
/l10n/l10n/nn-NO/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.7; previous revision: 1.6
done
Checking in pa-IN/browser/chrome/browser-region/region.properties;
/l10n/l10n/pa-IN/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.9; previous revision: 1.8
done
Checking in pl/browser/chrome/browser-region/region.properties;
/l10n/l10n/pl/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.25; previous revision: 1.24
done
Checking in pt-BR/browser/chrome/browser-region/region.properties;
/l10n/l10n/pt-BR/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.13; previous revision: 1.12
done
Checking in pt-PT/browser/chrome/browser-region/region.properties;
/l10n/l10n/pt-PT/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.9; previous revision: 1.8
done
Checking in ro/browser/chrome/browser-region/region.properties;
/l10n/l10n/ro/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.10; previous revision: 1.9
done
Checking in ru/browser/chrome/browser-region/region.properties;
/l10n/l10n/ru/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.24; previous revision: 1.23
done
Checking in si/browser/chrome/browser-region/region.properties;
/l10n/l10n/si/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.3; previous revision: 1.2
done
Checking in sk/browser/chrome/browser-region/region.properties;
/l10n/l10n/sk/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.8; previous revision: 1.7
done
Checking in sl/browser/chrome/browser-region/region.properties;
/l10n/l10n/sl/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.10; previous revision: 1.9
done
Checking in sq/browser/chrome/browser-region/region.properties;
/l10n/l10n/sq/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.7; previous revision: 1.6
done
Checking in sv-SE/browser/chrome/browser-region/region.properties;
/l10n/l10n/sv-SE/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.22; previous revision: 1.21
done
Checking in te/browser/chrome/browser-region/region.properties;
/l10n/l10n/te/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.3; previous revision: 1.2
done
Checking in th/browser/chrome/browser-region/region.properties;
/l10n/l10n/th/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.2; previous revision: 1.1
done
Checking in uk/browser/chrome/browser-region/region.properties;
/l10n/l10n/uk/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.9; previous revision: 1.8
done
Checking in zh-CN/browser/chrome/browser-region/region.properties;
/l10n/l10n/zh-CN/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.20; previous revision: 1.19
done
Checking in zh-TW/browser/chrome/browser-region/region.properties;
/l10n/l10n/zh-TW/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.17; previous revision: 1.16
done

Comment 10

9 years ago
Hi,

gu-IN also wants to add gmail to the list of mailto protocol handlers.
Let me knowk, if you want me to make this change myself OR you want to do it yourself.
(Assignee)

Comment 11

9 years ago
Hey Ankit, mind filing a new bug for gu-IN? Thanks.
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
(Assignee)

Comment 12

9 years ago
Oh, forgot to mention, landings on hg done, too.
(Assignee)

Updated

9 years ago
Blocks: 452199
Out of scope. Adding keyword to exclude.
QA Whiteboard: [QAnalyst-Triage?], [QAExclude]
Flags: needinfo?(jmercado)
QA Whiteboard: [QAnalyst-Triage?], [QAExclude] → [QAnalyst-Triage+][QAExclude]
Flags: needinfo?(jmercado)
You need to log in before you can comment on or make changes to this bug.