1292188 - Automatically retrieve l10n changesets from elmo

Status: RESOLVED FIXED

(Release Engineering :: Applications: Shipit, defect)

Description

[Rail Aliiev [:rail]]

This page contains links to l10n changesets for different products:

https://l10n.mozilla.org/shipping/drivers

Ship-it should:
* Fetch the data from elmo automatically (frontend or backend)
* Hide the l10n textarea and make it editable in advanced mode (just in case we have issues with the data)
* Remove the l10n dashboard check: bug 1290485
* Build2 to reuse build1 l10n data by default with ability to force refresh

Comment 1

[Johan Lorenzo [:jlorenzo]]

Attachment: PR

Comment 2

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8779730 [details] [review]
PR

> Ship-it should:
> * Fetch the data from elmo automatically (frontend or backend)
Done on the client side. I added tests for `getElmoUrl()`, but I haven't for `populateL10nChangesets()` which performs the async request and the UI manipulation. The easiest way to test it would be to use a mocking library (like sinon.js). At the first glance, I didn't want to increase the numbers of external libs, but if you're okay with me adding it, I'll be pleased to do so (probably in a follow up).

> * Hide the l10n textarea and make it editable in advanced mode (just in case we have issues with the data)
Done. The field remain disabled while the Ajax request is still pending.

> * Remove the l10n dashboard check: bug 1290485
Sylvestre has a patch there. I'll wait until this bug lands.

> * Build2 to reuse build1 l10n data by default with ability to force refresh
Discussed with :rail, it's better in a follow up. See bug 1294107.

Comment 3

[Johan Lorenzo [:jlorenzo]]

Landed at https://github.com/mozilla-releng/ship-it/commit/55265acb74d5a9dd2740a0b29397628b8231fb1e

Comment 4

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8779730 [details] [review]
PR

You are moving too fast for me!