1450571 - Remove translation code

Status: RESOLVED WONTFIX

(Firefox :: Translations, enhancement)

Description

[Tim Nguyen :ntim]

I believe the translation code is now abandoned and does not work. I could be wrong however...

There are add-ons which can provide this functionality.

Comment 1

[Florian Quèze [:florian]]

(In reply to Tim Nguyen :ntim from comment #0)
> I believe the translation code is now abandoned and does not work. I could
> be wrong however...

It's covered by automated tests, what makes you believe it doesn't work?

Comment 2

[Manish [:manishkk]]

Please let me know more about this bug in detail.
Thanks

Comment 3

[Tim Nguyen :ntim]

Attachment: Screenshot of broken translation feature (bing)

(In reply to Tim Nguyen :ntim from comment #0)
> I believe the translation code is now abandoned and does not work. I could
> be wrong however...
> 
> There are add-ons which can provide this functionality.

The page language detection (French) works fine, but this is what I get when I hit the translate button.

Comment 4

[Tim Nguyen :ntim]

Attachment: Screenshot of broken translation feature (Yandex)

I also tried with yandex as provider, and while language detection works, clicking translate also gives me an error.

Comment 5

[Manish [:manishkk]]

Where can I check for translation code? please guide me

Comment 6

[Tim Nguyen :ntim]

(In reply to Manish Kumar from comment #5)
> Where can I check for translation code? please guide me

We're not sure whether we want to remove it yet :) Once we'll have made a clear decision, I'll let you know.

The code is in browser/components/translation.

Comment 7

[:Felipe Gomes (needinfo for replies!)]

(In reply to Tim Nguyen :ntim from comment #4)
> Created attachment 8964204 [details]
> Screenshot of broken translation feature (Yandex)
> 
> I also tried with yandex as provider, and while language detection works,
> clicking translate also gives me an error.

FWIW, the language detector is done client-side, so it will always work. The translation is done server-side and requires a valid API key from either google or yandex, which we currently don't have.

Comment 8

[Florian Quèze [:florian]]

Tim, you are confirming that at least language detection and showing the translation bar works fine. To try the translation feature, you need to provide an API key for a translation service, like Felipe said in comment 7. Last I heard (that was in Q1), our BD team is still working on getting such an API key that we could distribute to users.

Comment 9

[Tim Nguyen :ntim]

Ok, I guess the lack of activity in the past months mislead me :(