My favorites | Sign in
Logo
                
New issue | Search
for
| Advanced search | Search tips
Issue 49: Remove "Download" link for each patch
2 people starred this issue and may be notified of changes. Back to list
Status:  Assigned
Owner:  john.abdelmalek
Type-Defect
Priority-Medium


Sign in to add a comment
 
Reported by john.abdelmalek, Sep 20, 2008
Chromium's Rietveld's instance is using about 100 MB of storage a week, and 
I'm looking at reducing that.

One thing I'm looking at it is reusing base files when they're the same as 
earlier patchsets.  This doesn't affect Rietveld's UI.

One other idea though is to remove the "Download" beside each file.  I 
don't think it's needed, since if someone wants to apply a patch they will 
most probably want to apply it to all the files.  If someone really wants 
to do it to just one file, then they can edit the full patch themself.  Are 
people ok with this change?
Comment 1 by m0.interactive, Mar 15, 2009
It seems okay to have this, I always wondered why someone needed to apply a patch for 
one file. It is usually for the whole changeset.
Comment 2 by gvanrossum, Mar 15, 2009
To be honest I'm not sure how removing the "Download patch" link saves you space...
Sign in to add a comment

Hosted by Google Code