912199 - Search engine setup for Firefox (desktop) for Catalan

Status: RESOLVED FIXED

(Mozilla Localizations :: ca / Catalan, defect)

Description

[Eduard Gamonal]

Attachment: Patch to update the search plugins

The search engine list needs to be updated.

The patch removes Llibres.cat (it doesn't work any more with opensearch) and Huubs (already removed in mobile #885079), and adds Twitter.

Comment 1

[Francesco Lodolo [:flod]]

Comment on attachment 799077 [details] [diff] [review]
Patch to update the search plugins

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

Patch tested, looks good to me.

Comment 2

[Eduard Gamonal]

Hi,
Can I do anything to help you?
It'd be very nice to have this ready for the next version. Users of the llibres.cat searchplugin get 404 errors.

Comment 3

[Francesco Lodolo [:flod]]

Comment on attachment 799077 [details] [diff] [review]
Patch to update the search plugins

Pike, do you have time to check this change (moving original r? to milos to f? to you)?

Comment 4

[Axel Hecht [:Pike]]

Comment on attachment 799077 [details] [diff] [review]
Patch to update the search plugins

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

r=me, too. Please land with a check-in comment referencing this bug and the reviews.

too bad that both sites just broke :-(

Comment 5

[Eduard Gamonal]

Hey, this has had no activity for three weeks but we can still make it for the next cycle.
How can we do it?

Comment 6

[Francesco Lodolo [:flod]]

Eduard, the patch is reviewed so you just need to check it in with a comment like "Bug 912199 - Search engine setup for Firefox (desktop) for Catalan, r=pike, flod".

If you prefer I can do it for you.

Comment 7

[Eduard Gamonal]

ah I thought you preferred that we don't do that. it's okay for me,I'll apply it, push it and mark this as resolved after the summit.
thanks!

Comment 8

[Francesco Lodolo [:flod]]

It's important you don't make changes without review, but once the patch is reviewed you can commit it, just reference bug+reviews to make l10n-drivers' work easier when checking sign-offs.

When you commit please put a link to the changeset in the bug too.

Comment 9

[Eduard Gamonal]

added in changeset 9092c6d1fbe8

http://hg.mozilla.org/releases/l10n/mozilla-aurora/ca/file/9092c6d1fbe8