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

Issue 4889997257736192: Issue 1821 - Add product support URL to installer (Closed)

Created:
March 13, 2015, 1:15 p.m. by Oleksandr
Modified:
June 1, 2015, 9:27 a.m.
Reviewers:
sergei, Eric
Visibility:
Public.

Description

Issue 1821 - Add product support URL to installer

Patch Set 1 #

Total comments: 3

Patch Set 2 : Removing comment #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M installer/src/msi/adblockplusie.wxs View 1 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 5
Oleksandr
March 13, 2015, 1:16 p.m. (2015-03-13 13:16:44 UTC) #1
Eric
Looks fine overall. http://codereview.adblockplus.org/4889997257736192/diff/5629499534213120/installer/src/msi/adblockplusie.wxs File installer/src/msi/adblockplusie.wxs (right): http://codereview.adblockplus.org/4889997257736192/diff/5629499534213120/installer/src/msi/adblockplusie.wxs#newcode154 installer/src/msi/adblockplusie.wxs:154: --> Can we kill this comment ...
March 13, 2015, 4:37 p.m. (2015-03-13 16:37:11 UTC) #2
Oleksandr
http://codereview.adblockplus.org/4889997257736192/diff/5629499534213120/installer/src/msi/adblockplusie.wxs File installer/src/msi/adblockplusie.wxs (right): http://codereview.adblockplus.org/4889997257736192/diff/5629499534213120/installer/src/msi/adblockplusie.wxs#newcode159 installer/src/msi/adblockplusie.wxs:159: <Property Id="ARPHELPLINK" Value="https://adblockplus.org/en/bugs"/> On 2015/03/13 16:37:11, Eric wrote: > ...
March 20, 2015, 4:44 a.m. (2015-03-20 04:44:33 UTC) #3
Eric
Patch set 2 LGTM.
March 20, 2015, 12:14 p.m. (2015-03-20 12:14:19 UTC) #4
sergei
March 30, 2015, 9:41 a.m. (2015-03-30 09:41:23 UTC) #5
LGTM

Powered by Google App Engine
This is Rietveld