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

Issue 29660578: Gitlab #2 - Extend privacy policy for newsletter signup form (Closed)

Created:
Jan. 9, 2018, 2:42 p.m. by juliandoucette
Modified:
Jan. 31, 2018, 7:03 p.m.
Reviewers:
ire
CC:
wspee
Base URL:
https://hg.adblockplus.org/web.eyeo.com
Visibility:
Public.

Description

Issue : https://gitlab.com/eyeo/web.eyeo.com/issues/2 Spec : https://gitlab.com/eyeo/spec/merge_requests/109

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -1 line) Patch
M pages/privacy.html View 2 chunks +20 lines, -1 line 2 comments Download

Messages

Total messages: 6
juliandoucette
Jan. 9, 2018, 2:42 p.m. (2018-01-09 14:42:56 UTC) #1
juliandoucette
(I know I could have created a pull request on Gitlab. I was lazy / ...
Jan. 9, 2018, 2:44 p.m. (2018-01-09 14:44:53 UTC) #2
juliandoucette
@wspee is this review premature? I would LGTM + NITs the spec pull request. Except ...
Jan. 9, 2018, 2:51 p.m. (2018-01-09 14:51:13 UTC) #3
ire
On 2018/01/09 14:44:53, juliandoucette wrote: > (I know I could have created a pull request ...
Jan. 10, 2018, 9:58 a.m. (2018-01-10 09:58:53 UTC) #4
ire
Note: I also left a couple of comments on the spec PR. https://codereview.adblockplus.org/29660578/diff/29660579/pages/privacy.html File pages/privacy.html ...
Jan. 10, 2018, 9:59 a.m. (2018-01-10 09:59:31 UTC) #5
juliandoucette
Jan. 11, 2018, 12:56 p.m. (2018-01-11 12:56:10 UTC) #6
I'll update as soon as Judith approves your changes in the PR. Thank you!

https://codereview.adblockplus.org/29660578/diff/29660579/pages/privacy.html
File pages/privacy.html (right):

https://codereview.adblockplus.org/29660578/diff/29660579/pages/privacy.html#...
pages/privacy.html:138: <h3 id="newsletter">
On 2018/01/10 09:59:31, ire wrote:
> This should be an <h4>

Good catch.

Powered by Google App Engine
This is Rietveld