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

Issue 29370756: Issue 4280 - Include sub-domains for xg4ken.com click redirects (Closed)

Created:
Jan. 5, 2017, 4:21 p.m. by arthur
Modified:
Jan. 5, 2017, 4:54 p.m.
Reviewers:
kzar
CC:
Felix Dahlke, Sebastian Noack
Visibility:
Public.

Description

Issue 4280 - Include sub-domains for xg4ken.com click redirects

Patch Set 1 #

Total comments: 1

Patch Set 2 : Removed obsolete domain #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -3 lines) Patch
M easylist+exceptionrules_content_blocker.json View 2 chunks +1 line, -3 lines 0 comments Download

Messages

Total messages: 4
arthur
Hey Dave, As discussed in the e-mail. Thanks, Arthur
Jan. 5, 2017, 4:26 p.m. (2017-01-05 16:26:38 UTC) #1
kzar
https://codereview.adblockplus.org/29370756/diff/29370757/easylist+exceptionrules_content_blocker.json File easylist+exceptionrules_content_blocker.json (left): https://codereview.adblockplus.org/29370756/diff/29370757/easylist+exceptionrules_content_blocker.json#oldcode655882 easylist+exceptionrules_content_blocker.json:655882: "www.myvisualiq.net", Isn't there still www.xg4ken.com below this line?
Jan. 5, 2017, 4:30 p.m. (2017-01-05 16:30:49 UTC) #2
arthur
On 2017/01/05 16:30:49, kzar wrote: > https://codereview.adblockplus.org/29370756/diff/29370757/easylist+exceptionrules_content_blocker.json > File easylist+exceptionrules_content_blocker.json (left): > > https://codereview.adblockplus.org/29370756/diff/29370757/easylist+exceptionrules_content_blocker.json#oldcode655882 > ...
Jan. 5, 2017, 4:39 p.m. (2017-01-05 16:39:57 UTC) #3
kzar
Jan. 5, 2017, 4:44 p.m. (2017-01-05 16:44:16 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld