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

Issue 29613772: Issue 6042 - Introduce ui_warn_tracking boolean preference and updated adbockplusui dependency to... (Closed)

Created:
Nov. 21, 2017, 3:15 p.m. by saroyanm
Modified:
Nov. 22, 2017, 12:02 p.m.
Reviewers:
Thomas Greiner, kzar
Visibility:
Public.

Description

Issue 6042 - Introduced ui_warn_tracking boolean preference and updated adbockplusui dependency to bb9b1b67aedf9

Patch Set 1 : #

Patch Set 2 : Updated the dependency of adblockplusui #

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

Messages

Total messages: 8
saroyanm
@greiner maybe you can also give your final go regarding the description as I updated ...
Nov. 21, 2017, 3:27 p.m. (2017-11-21 15:27:40 UTC) #1
kzar
LGTM but it might make sense to wait until the adblockplusui changes are ready and ...
Nov. 21, 2017, 3:44 p.m. (2017-11-21 15:44:47 UTC) #2
saroyanm
On 2017/11/21 15:44:47, kzar wrote: > LGTM but it might make sense to wait until ...
Nov. 21, 2017, 3:46 p.m. (2017-11-21 15:46:52 UTC) #3
Thomas Greiner
On 2017/11/21 15:27:40, saroyanm wrote: > @greiner maybe you can also give your final go ...
Nov. 21, 2017, 5:40 p.m. (2017-11-21 17:40:35 UTC) #4
saroyanm
Updated the dependency, will wait for the final go from Dave and push the changes ...
Nov. 22, 2017, 10:48 a.m. (2017-11-22 10:48:21 UTC) #5
kzar
On 2017/11/22 10:48:21, saroyanm wrote: > Updated the dependency, will wait for the final go ...
Nov. 22, 2017, 10:55 a.m. (2017-11-22 10:55:39 UTC) #6
saroyanm
On 2017/11/22 10:55:39, kzar wrote: > On 2017/11/22 10:48:21, saroyanm wrote: > > Updated the ...
Nov. 22, 2017, 11:35 a.m. (2017-11-22 11:35:14 UTC) #7
kzar
Nov. 22, 2017, 11:38 a.m. (2017-11-22 11:38:18 UTC) #8
Thanks LGTM, go ahead and push this when you're ready.

Powered by Google App Engine
This is Rietveld