Two blank lines between these.

Trailing whitespace should be removed.

Also, imports are alphabetical, so the filemanager one should go before the reviews ones.

These should be combined, since one is overriding the other.

Two blank lines between these.

Same comment about the filemanager one. Should be alphabetical.

Two blank lines between impors and classes.

First line, immediately following the """, should be a one-line summary of the form, followed by a blank line, and then an optional description.

Alignment issues.

I don't think we want draft_caption set by default.

Two blank lines.

Same comment about docs.

Should be spaces after :

Also, this looks like it may be > 80 characters. If so, wrap widget= onto the next line.

No blank line here.

Alignment issues.

No blank line between these.

reviews.errors should go before reviews.signals.

Two blank lines.

Should be on the same line (summary).

Same line.

Should just be return self.upfile.url.

Same line.

This should probably be:

return os.path.basename(self.upfile.name)

return self.caption

This should use the @permalink decorator. See other instances of get_absolute_url, specifically Screenshot's.

This view (and therefore the file) can go away, since the API should be used for this.

Two blank lines.

Two blank lines.

Two blank lines.

Wrong name. deleteFile.

Wrong path. Shouldn't be screenshots.

screenshot is wrong.

Alignment problems.

Two blank lines.

on_ready.apply(this, arguments);

on_done.apply(this, arguments);

on_done.apply(this, arguments);

on_done.apply(this, arguments);

Two blank lines.

on_ready.apply(this, arguments);

Best as:

if (this.text == "") {
    this.deleteComment();
}

on_done.apply(this, arguments);

on_done.apply(this, arguments);

on_done.apply(this, arguments);

Space after the ,

Blank line before this.

Blank line before this.

UploadedFile should be on the next line, aligned with the other parameters.

Same line.

Alignment issues.

I'd prefer "uploaded_file"

This can be better written as:

if context_type in ('comment', 'screenshot_comment', 'file_comment'):

Won't be needed once the code is updated to use HTTP DELETE instead of this view.

This will all need to be updated for the HTTP DELETE change. You can model it after how screenshots now do it.

Trailing whitespace.

This was copy/pasted from broken code. Look at what the above handler for screenshots now does.

forms before models.

Two blank lines.

More can fit on that first line.

Needs @webapi_check_local_site before the other decorators.

Needs NOT_LOGGED_IN in the list.

Should return _no_access_error(request.user)

Needs @webapi_check_local_site before other decorators.

Also needs a @webapi_response_errors(DOES_NOT_EXIST, NOT_LOGGED_IN, PERMISSION_DENIED)

Should return _no_access_error(request.user)

@webapi_check_local_site

@webapi_check_local_site

@webapi_check_local_site

Blank lines should remain.

Blank lines should remain.

No blank lines.

@webapi_check_local_site

@webapi_check_local_site

Two blank lines.

@webapi_check_local_site and @webapi_response_errors.

Can fit on one line.

return _no_access_error(request.user)

Alignment issues.

@webapi_check_local_site

There's a pattern here :) Going to stop saying it now. Can you go through and make sure all others have it too?

Should include NOT_LOGGED_IN.

Same with every other @webapi_response_errors containing PERMISSION_DENIED.

_no_access_error

Same with all others.

Two blank lines.

Hi Christian,

Sorry it took so long to reply-- it was crunch time for one of my classes. 
I've pushed the changes you mentioned to my github.

Thanks,
Laila