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

Issue 4652755411009536: Issue #1832 - Updated public suffixes to account for registration of the new ".uk" domains. (Closed)

Created:
Jan. 20, 2015, 1:13 p.m. by kzar
Modified:
Jan. 21, 2015, 10:32 a.m.
Reviewers:
Sebastian Noack
Visibility:
Public.

Description

Issue #1832 - Updated public suffixes to account for registration of the new ".uk" domains.

Patch Set 1 #

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

Messages

Total messages: 4
kzar
Jan. 20, 2015, 1:14 p.m. (2015-01-20 13:14:42 UTC) #1
Sebastian Noack
I think there should be an issue for this. And another issue (that can be ...
Jan. 20, 2015, 1:21 p.m. (2015-01-20 13:21:06 UTC) #2
Sebastian Noack
LGTM
Jan. 20, 2015, 1:39 p.m. (2015-01-20 13:39:33 UTC) #3
kzar
Jan. 20, 2015, 1:45 p.m. (2015-01-20 13:45:32 UTC) #4
On 2015/01/20 13:39:33, Sebastian Noack wrote:
> LGTM

Thanks, just had a go at writing the issue for the conversion script as well.
(Might need some work though) https://issues.adblockplus.org/ticket/1834

Powered by Google App Engine
This is Rietveld