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

Issue 29560569: Issue 5815 - Fixed Crowdin upload with non-ASCII characters (Closed)

Created:
Oct. 1, 2017, 2:37 a.m. by Sebastian Noack
Modified:
Oct. 1, 2017, 2:40 p.m.
Visibility:
Public.

Description

Issue 5815 - Fixed Crowdin upload with non-ASCII characters

Patch Set 1 #

Total comments: 4

Patch Set 2 : Put each argument on seperate line #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -3 lines) Patch
M localeTools.py View 1 2 chunks +4 lines, -3 lines 0 comments Download

Messages

Total messages: 5
Sebastian Noack
https://codereview.adblockplus.org/29560569/diff/29560570/localeTools.py File localeTools.py (right): https://codereview.adblockplus.org/29560569/diff/29560570/localeTools.py#newcode356 localeTools.py:356: data=data.encode('utf-8'), data is a unicode object. When passing any ...
Oct. 1, 2017, 2:42 a.m. (2017-10-01 02:42:07 UTC) #1
Vasily Kuznetsov
Hi Sebastian, One little suggestion below but I don't feel strongly about it, so LGTM ...
Oct. 1, 2017, 9:54 a.m. (2017-10-01 09:54:28 UTC) #2
tlucas
Hey guys, LGTM
Oct. 1, 2017, 10:41 a.m. (2017-10-01 10:41:25 UTC) #3
Sebastian Noack
https://codereview.adblockplus.org/29560569/diff/29560570/localeTools.py File localeTools.py (right): https://codereview.adblockplus.org/29560569/diff/29560570/localeTools.py#newcode355 localeTools.py:355: ).format(boundary=boundary, name=name, On 2017/10/01 09:54:28, Vasily Kuznetsov wrote: > ...
Oct. 1, 2017, 2:20 p.m. (2017-10-01 14:20:54 UTC) #4
Vasily Kuznetsov
Oct. 1, 2017, 2:28 p.m. (2017-10-01 14:28:11 UTC) #5
even more LGTM ;)

Powered by Google App Engine
This is Rietveld