Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1147)

Issue 6197071: Add an endpoint to download the binary content of a patch (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 11 months ago by Roger Tawa
Modified:
1 year, 11 months ago
Reviewers:
M-A
Visibility:
Public.

Description

Add an endpoint to download the binary content of a patch

http://code.google.com/p/rietveld/source/detail?r=d9f35f2ee678d75e00606b971a0...

Patch Set 1 #

Total comments: 1

Patch Set 2 : Remove cache directive #

Unified diffs Side-by-side diffs Delta from patch set Stats (+44 lines, -1 line) Patch
M codereview/urls.py View 1 chunk +1 line, -0 lines 0 comments Download
M codereview/views.py View 1 chunk +1 line, -1 line 0 comments Download
M codereview/views_chromium.py View 1 3 chunks +42 lines, -0 lines 0 comments Download

Messages

Total messages: 3
Roger Tawa
Salut MA, Please take a look. I wonder if this change is really needed now ...
1 year, 11 months ago #1
M-A
lgtm http://codereview.appspot.com/6197071/diff/1/codereview/views_chromium.py File codereview/views_chromium.py (right): http://codereview.appspot.com/6197071/diff/1/codereview/views_chromium.py#newcode520 codereview/views_chromium.py:520: response['Cache-Control'] = 'no-cache, no-store' Not sure about the ...
1 year, 11 months ago #2
Roger Tawa
1 year, 11 months ago #3
Remove cache directive
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1278:e6ce13d99bf5