1165120 - Locale support for suggested sites

Status: RESOLVED FIXED

(Firefox for iOS :: General, defect)

Description

[Wesley Johnston (:wesj)]

The suggested sites list we ship is in English. We should make it localizable (and allow shipping different sites in other locales).

Comment 1

[Wesley Johnston (:wesj)]

Is this going to be possible? Right now I just checked in a static list here (from Fennec with the AMO and Marketplace links removed):

https://github.com/mozilla/firefox-ios/blob/master/Client/Assets/suggestedsites.json

while I waited to see what the search engines group did. Sounds like we can do something similar to what they did here?

Comment 2

[Francesco Lodolo [:flod]]

http://hg.mozilla.org/mozilla-central/file/tip/mobile/locales/en-US/chrome/region.properties#l42

We're not currently localizing suggested sites at the moment, so I don't really have an answer to this question.

Technically they're localizable in region.properties, practically we never implemented any policy (what to accept, etc.) nor started accepting localizations for the ones shipping in en-US.

Comment 3

[Axel Hecht [:Pike]]

Darren, do content services have plans for suggested tiles on iOS? Also looping in mconnor.

I'm not convinced that we should tie the suggested sites to our update cycle.

Do we have any existing data processes to update iOS fx with server-provided data snippets?

Comment 4

[Wesley Johnston (:wesj)]

Currently there's not much infrastructure for pulling down snippets at runtime (we do pull from a local url, which should also work for a remote one, but that about it).

I assume we will sometime, but its not for v1.

Comment 5

[Darren Herman]

@axel we currently do not have this on the roadmap.  Maybe in the future, but right now, Content Services is focused on desktop.

Comment 6

[Wesley Johnston (:wesj)]

Going to make sure that the redirects work right right now. Also going to update the SUMO link to the iOS page in this bug.

Comment 7

[Wesley Johnston (:wesj)]

Attachment: Update urls

This updates the urls to be local agnostic. It doesn't fix the titles though :(

Comment 8

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Comment on attachment 8624770 [details] [review]
Update urls

LGTM