Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29931199: Issue 7024 - Renamed az translation to az-latn in Microsoft Edge builds (Closed)

Created:
Oct. 29, 2018, 10:25 p.m. by Sebastian Noack
Modified:
Oct. 29, 2018, 10:38 p.m.
Reviewers:
tlucas, geo
Visibility:
Public.

Description

Issue 7024 - Renamed az translation to az-latn in Microsoft Edge builds

Patch Set 1 #

Patch Set 2 : Properly handle if translation doesn't exist #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -0 lines) Patch
M packagerEdge.py View 1 1 chunk +8 lines, -0 lines 0 comments Download

Messages

Total messages: 4
Sebastian Noack
Oct. 29, 2018, 10:25 p.m. (2018-10-29 22:25:59 UTC) #1
tlucas
Is that seriously the solution to our problem? :D LGTM
Oct. 29, 2018, 10:33 p.m. (2018-10-29 22:33:54 UTC) #2
geo
On 2018/10/29 22:25:59, Sebastian Noack wrote: Just a small typo in description and title, `at-latn`, ...
Oct. 29, 2018, 10:34 p.m. (2018-10-29 22:34:30 UTC) #3
Sebastian Noack
Oct. 29, 2018, 10:37 p.m. (2018-10-29 22:37:55 UTC) #4
On 2018/10/29 22:34:30, geo wrote:
> On 2018/10/29 22:25:59, Sebastian Noack wrote:
> 
> Just a small typo in description and title, `at-latn`, other than that LGTM.

Oops, I fixed the title and description in the review, but forgot to change the
commit message before pushing. :(

Powered by Google App Engine
This is Rietveld