Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1231)

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

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 months, 3 weeks ago by Sebastian Noack
Modified:
8 months, 3 weeks ago
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
8 months, 3 weeks ago (2018-10-29 22:25:59 UTC) #1
tlucas
Is that seriously the solution to our problem? :D LGTM
8 months, 3 weeks ago (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`, ...
8 months, 3 weeks ago (2018-10-29 22:34:30 UTC) #3
Sebastian Noack
8 months, 3 weeks ago (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. :(
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 87257f5