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

Issue 29749597: Fixes #17 - Added Microsoft Edge to requirements page (Closed)

Created:
April 11, 2018, 9:17 p.m. by juliandoucette
Modified:
April 16, 2018, 12:10 p.m.
Reviewers:
ire
Base URL:
https://hg.adblockplus.org/web.adblockplus.org
Visibility:
Public.

Patch Set 1 #

Total comments: 2

Patch Set 2 : Addressed #2 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M pages/requirements.tmpl View 1 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 5
juliandoucette
April 11, 2018, 9:17 p.m. (2018-04-11 21:17:47 UTC) #1
ire
Thanks Julian! https://codereview.adblockplus.org/29749597/diff/29749598/pages/requirements.tmpl File pages/requirements.tmpl (right): https://codereview.adblockplus.org/29749597/diff/29749598/pages/requirements.tmpl#newcode42 pages/requirements.tmpl:42: </li>{{ "Microsoft Edge released with Windows 10 ...
April 12, 2018, 3:17 p.m. (2018-04-12 15:17:20 UTC) #2
juliandoucette
Oops! Thanks Ire! New patchset up!
April 12, 2018, 3:24 p.m. (2018-04-12 15:24:27 UTC) #3
ire
LGTM
April 12, 2018, 3:47 p.m. (2018-04-12 15:47:31 UTC) #4
juliandoucette
April 16, 2018, 12:10 p.m. (2018-04-16 12:10:37 UTC) #5

Powered by Google App Engine
This is Rietveld