1103229 - Remove affiliate codes from default Google plugin in ku locale

Status: RESOLVED WONTFIX

(Mozilla Localizations :: ku / Kurdish, defect, P1)

Description

[Kev Needham [:kev]]

Attachment: Remove affliate codes from Google KU plugin

Per expiration of the search agreement, the Google affiliate codes must be removed from any and all official builds. Note that Google plugins also exist in the JP and KU locales, and also need to be modified.

Comment 1

[:Gavin Sharp [email: [e-mail]]]

Comment on attachment 8527059 [details] [diff] [review]
Remove affliate codes from Google KU plugin

We should probably limit this to strictly the agreement-related parameter changes (and split the icon/name changes to some other bug).

Flod should probably advise on the mechanics for this.

Comment 2

[Kev Needham [:kev]]

I just wanted to make it consistent with our default, which it wasn't before, and add the hl=en parameter. I can re-submit with just the changes to the agreement parameters if necessary, and I guess I kind of mangled the patch. I'll re-submit after dinner.

Comment 3

[Francesco Lodolo [:flod]]

Mechanics: when we agree on a patch, I'll be glad to land it on all branches. 

Pending a question about sign-offs: current last sign-off for Kurdish is for fx28, I'm not sure if we're in shape to take an updated sign-off next cycle to just update the searchplugin (i.e. there might be errors in the localization that we don't want to push out).

And I think that we should try to clarify if we still need a Kurdish version of Google (originally bug 449112). Maybe it's not needed anymore and we can rely on the en-US version like other locales.
Or maybe just drop the codes and then figure it out in a follow-up.

Firefox 34 does not seem likely given the timeline (you can push the change to beta, but it needs a sign-off to get in the build).

Comment 4

[Francesco Lodolo [:flod]]

Comment on attachment 8527059 [details] [diff] [review]
Remove affliate codes from Google KU plugin

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

Having said that, the changes look sane compared to en-US from bug 1103216.

Comment 5

[Axel Hecht [:Pike]]

I know that Jeff and I have been talking about stopping Kurdish for the lack of feedback, and updates.

Jeff, should we just stop shipping Kurdish now?

Comment 6

[Jeff Beatty [:gueroJeff]]

Yes. After a couple of months worth of attempts to reach out to the current team and find replacememnt localizers, I've been unsuccessful. We should start the EOL with this next cycle.

Comment 7

[Axel Hecht [:Pike]]

'k. flagging needinfo for Lawrence to remove Kurdish for 34+. We'll need more patches for this, filing more bugs.

We'll also need an update to bug 1098453.

Comment 8

[Lawrence Mandel [:lmandel] (use needinfo)]

Given that we're going to take bug 1103991 in 34, I don't think there is a reason to land this change on Beta. We can land changes on 35+ if it is desirable to try to resurrect this locale at some point.

Comment 9

[Lukas Blakk [:lsblakk] use ?needinfo]

Not tracking for 35 since we've stopped builds.