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

Issue 29624558: Fixed #4929 - Rename App from Adblock to Adblock Browser (Closed)

Created:
Nov. 29, 2017, 1:22 p.m. by a.shephard
Modified:
Jan. 5, 2018, 2:41 p.m.
Reviewers:
d108, dean
Visibility:
Public.

Description

Fixed #4929 - Rename App from Adblock to Adblock Browser

Patch Set 1 #

Patch Set 2 : Fixed #4929 - Rename app from Adblock to Adblock Browser #

Patch Set 3 : Fixed #4929 - Rename app from Adblock to Adblock Browser #

Patch Set 4 : Fixed #4929 - Rename app from Adblock to Adblock Browser #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M AdBlockBrowser/Info.plist View 1 2 3 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6
a.shephard
Nov. 29, 2017, 1:24 p.m. (2017-11-29 13:24:30 UTC) #1
dean
On 2017/11/29 13:24:30, a.shephard wrote: LGTM, but whats with all the indents? Did Xcode auto-format ...
Nov. 30, 2017, 10:54 a.m. (2017-11-30 10:54:58 UTC) #2
a.shephard
On 2017/11/30 10:54:58, dean wrote: > On 2017/11/29 13:24:30, a.shephard wrote: > > LGTM, but ...
Nov. 30, 2017, 11:18 a.m. (2017-11-30 11:18:45 UTC) #3
a.shephard
Patchset 3. Ignore 2, accidentally compared against develop branch rather than the feature branch.
Dec. 4, 2017, 3:32 p.m. (2017-12-04 15:32:19 UTC) #4
dean
On 2017/12/04 15:32:19, a.shephard wrote: > Patchset 3. Ignore 2, accidentally compared against develop branch ...
Dec. 5, 2017, 2:31 p.m. (2017-12-05 14:31:30 UTC) #5
d108
Jan. 5, 2018, 1:42 a.m. (2018-01-05 01:42:28 UTC) #6
LGTM

Powered by Google App Engine
This is Rietveld