787362 - Discount first level changes on L10n KPI dashboard

Status: RESOLVED FIXED

(support.mozilla.org :: Knowledge Base Software, task, P2)

Description

[Kadir Topal [:atopal]]

The L10n coverage chart on the KPI dashboard currently takes into account trivial changes (first level) and counts localizations as not up to date as long as they don't incorporate those changes. Since we agree that first level changes have no meaningful effect on localization they should be discarded when measuring the L10n coverage.

Comment 1

[Ricky Rosario [:rrosario, :r1cky]]

CCing Ibai. I remember talking about this during the development of the algorithm. There was a reason we did it as is.

Comment 2

[Kadir Topal [:atopal]]

Just for context, this was a result from yesterdays KB meeting.

Comment 3

[:ibai]

So the original bug is https://bugzilla.mozilla.org/show_bug.cgi?id=727084

There was a misunderstanding from my side at the end of the bug. Because we don't request the first level changes to be localized we should discount them from the metric. 

There doesn't seem to be a reason to keep those en-US updates as part of the other locale's out-dated articles.

Comment 4

[Ricky Rosario [:rrosario, :r1cky]]

I don't think this should take more than a half day. Keep in mind that we can't go back in time to recalculate the old data. => 1pt

Comment 5

[Ricky Rosario [:rrosario, :r1cky]]

Grabbing.

Comment 6

[Ricky Rosario [:rrosario, :r1cky]]

Pull request:
https://github.com/mozilla/kitsune/pull/892

Landed on master:
https://github.com/mozilla/kitsune/commit/1b3e0475228c200fd9a4b77390439b3221c24e8c

Comment 7

[Ricky Rosario [:rrosario, :r1cky]]

Deployed to prod.

Comment 8

[Will Kahn-Greene [:willkg] ET needinfo? me]

Adding s= sprint information to whiteboard.