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

Issue 29322993: Issue 2677 - Reset subscriptions.xml to serve as mere test data (Closed)

Created:
July 29, 2015, 4:32 p.m. by Thomas Greiner
Modified:
July 30, 2015, 9:31 a.m.
Reviewers:
saroyanm
CC:
Felix Dahlke
Visibility:
Public.

Description

I clarified in the README that the included subscriptions.xml contains test data and should not be imported. I also got rid of the non-standard data in the local subscriptions.xml so that the ones from the platform versions can be used in its stead. That change is also in-line with the upcoming changes described in #2668.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -21 lines) Patch
M README.md View 1 chunk +2 lines, -0 lines 0 comments Download
M options.js View 1 chunk +2 lines, -2 lines 0 comments Download
M subscriptions.xml View 2 chunks +7 lines, -19 lines 0 comments Download

Messages

Total messages: 2
Thomas Greiner
July 29, 2015, 4:37 p.m. (2015-07-29 16:37:43 UTC) #1
saroyanm
July 29, 2015, 4:59 p.m. (2015-07-29 16:59:02 UTC) #2
On 2015/07/29 16:37:43, Thomas Greiner wrote:

LGTM

Powered by Google App Engine
This is Rietveld