415585 - (fx3-l10n-el) Firefox 3 el release tracker

Status: RESOLVED FIXED

(Mozilla Localizations :: el / Greek, defect)

Description

[Axel Hecht [:Pike]]

This is a tracker bug for releasing Firefox 3 el.

This bug is not that detailed, but as we get particular work items, they should block this bug for tracking, and better discoverability.

Comment 1

[Mic Berman]

Kostas
will you be making any changes to the search plug-ins, live bookmark or RSS readers?

Comment 2

[Kostas Papadimas]

Yes, 
I would like to change the default  English  live bookmark ( http://newsrss.bbc.co.uk/rss/newsonline_world_edition/front_page/rss.xml ) with the Greek news rss from google (http://news.google.gr/nwshp?hl=el&tab=wn&output=rss)  in order to provide local news (unfortunately BBC world servide does offers any more news in Greek).
Is this possible?

Comment 3

[Mic Berman]

yes, totally, moving this discussion comment 1 and 2 to another bug for web services, will block this bug

Comment 4

[Mic Berman]

clarifying yes in comment #3 above, yes = we can do this as we want local language live bookmark and we do not need explicit permission from google for this news feed so we are good to go

Comment 5

[Axel Hecht [:Pike]]

There is no dedicated web-services bug for Greek, I assume we're going with en-US defaults for now? If so, we'd probably still need yahoo mail to be included in region.properties.

Comment 6

[Kostas Papadimas]

(In reply to comment #5)
> There is no dedicated web-services bug for Greek, I assume we're going with
> en-US defaults for now?

Yes en-US defaults are fine...

>If so, we'd probably still need yahoo mail to be
> included in region.properties.
OK .. Added yahoo mail  (http://l10n.mozilla.org/buildbot/changes/7594 ) in region.properties

Comment 7

[Axel Hecht [:Pike]]

As mentioned in the comment, you need to bump the defaultHandlersVersion when making changes, otherwise they won't get picked up.

Comment 8

[Kostas Papadimas]

Attachment: region-properties

Is this OK?

Comment 9

[Axel Hecht [:Pike]]

Comment on attachment 314522 [details] [diff] [review]
region-properties

Post-mortem review. Patches like this, and the yahoo fix, should really be reviewed before checking in, that would have caught this bug.

Comment 10

[Kostas Papadimas]

(In reply to comment #9)
> (From update of attachment 314522 [details] [diff] [review])
> Post-mortem review. Patches like this, and the yahoo fix, should really be
> reviewed before checking in, that would have caught this bug.
> 

Actually el/region-properties is exactly the same as en-US/region-properties. Why I should add more "noise" asking for a review?  Ideally, compare-locales should catch the missing yahoo strings (but this would cause problems to other locales) (and if I was more carefull reading the comment for bumping the defaultHandlersVersion ...) 

Anyway, sorry for the trouble..

Kostas

Comment 11

[Axel Hecht [:Pike]]

FIXED, we shipped with fx 3.0.