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

Issue 955: Allow upload with no base (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 months, 1 week ago by Raghu
Modified:
2 months ago
Reviewers:
GvR, ondrej.certik
SVN Base:
http://rietveld.googlecode.com/svn/trunk/

Description

I am closing this as Ondrej's patch (1026) includes code from this one.

Patch Set 1

Total comments: 3

Patch Set 2 : I incorporated the changes as per your comments in previous patch.

Total comments: 2
Raw unified diffs Stats Side-by-side diffs with inline comments Delta from patch set
codereview/models.py 1 chunk 13 lines 1 comment 1
codereview/views.py 2 chunks 32 lines 1 comment
static/upload.py 3 chunks 37 lines 0 comments 1

Messages

Total messages: 6
GvR
http://codereview.appspot.com/955/diff/1/3 File codereview/models.py (right): http://codereview.appspot.com/955/diff/1/3#newcode65 Line 65: base = db.StringProperty() Why do you need that? ...
2 months, 1 week ago
Raghu
2 months, 1 week ago
Raghu
I should mention that this code should be treated as only prototype. That is the ...
2 months, 1 week ago
GvR
Are you asking for feedback? This code seems to be too incomplete to be able ...
2 months, 1 week ago
Raghu
> Are you asking for feedback? This code seems to be too > incomplete to ...
2 months, 1 week ago
ondrej.certik
2 months, 1 week ago
> Sorry. I merely intended this as a starting point for Ondrej > or who ever
else
> wanting to implement Plan B (as described on 
> codereview-discuss list). 

Thanks for your work! I based my upload base files patch on yours here:

http://codereview.appspot.com/1026
Sign in to reply to this message.

Powered by Google App Engine
This is Rietveld r168