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

Issue 29363604: Noissue - Fix typo in disable auto updates switch (Closed)

Created:
Nov. 21, 2016, 10:19 a.m. by sergei
Modified:
Nov. 21, 2016, 2:06 p.m.
Reviewers:
Oleksandr
CC:
Felix Dahlke, anton
Visibility:
Public.

Description

review: https://codereview.adblockplus.org/29363604/

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M lib/updater.js View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3
sergei
Nov. 21, 2016, 10:20 a.m. (2016-11-21 10:20:54 UTC) #1
Oleksandr
Ouch. Nice catch! LGTM, only the commit message could mention disabling auto updates, I think. ...
Nov. 21, 2016, 10:30 a.m. (2016-11-21 10:30:29 UTC) #2
sergei
Nov. 21, 2016, 10:32 a.m. (2016-11-21 10:32:55 UTC) #3
On 2016/11/21 10:30:29, Oleksandr wrote:
> Ouch. Nice catch! ...
Thanks to anton.

Commit title is updated.

Powered by Google App Engine
This is Rietveld