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

Issue 29531664: Noissue - Added missing languages to abp.org (Closed)

Created:
Aug. 30, 2017, 12:18 p.m. by juliandoucette
Modified:
Aug. 30, 2017, 12:28 p.m.
Reviewers:
tamara
CC:
saroyanm
Base URL:
https://hg.adblockplus.org/web.adblockplus.org
Visibility:
Public.

Description

Noissue - Added missing languages to abp.org

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -1 line) Patch
M settings.ini View 1 chunk +5 lines, -1 line 2 comments Download

Messages

Total messages: 4
juliandoucette
Aug. 30, 2017, 12:18 p.m. (2017-08-30 12:18:38 UTC) #1
juliandoucette
https://codereview.adblockplus.org/29531664/diff/29531665/settings.ini File settings.ini (right): https://codereview.adblockplus.org/29531664/diff/29531665/settings.ini#newcode26 settings.ini:26: cs = čeština Is this supposed to be capitalized ...
Aug. 30, 2017, 12:19 p.m. (2017-08-30 12:19:59 UTC) #2
tamara
On 2017/08/30 12:18:38, juliandoucette wrote: LGTM
Aug. 30, 2017, 12:20 p.m. (2017-08-30 12:20:37 UTC) #3
tamara
Aug. 30, 2017, 12:21 p.m. (2017-08-30 12:21:45 UTC) #4
On 2017/08/30 12:20:37, tamara wrote:
> On 2017/08/30 12:18:38, juliandoucette wrote:
> 
> LGTM

Crap, didn't see your second message. Nope, Czech's language name in Czech isn't
capitalised.

Powered by Google App Engine
This is Rietveld