239: Attach files other than screenshots

regis.d*******@gmai***** (Google Code) (Is this you? Claim this profile.)
chipx86
chipx86
May 15, 2011
382, 1157, 1169, 1228
This is a feature request.

I would like to be able to attach a file (other than a screenshot) to a review request.

For example, today I wanted to attach a small C file that was the test case that I used to test my 
change.

For me, an idela interface would be:
Click on "Attach file"
Popup with the file path field+browse button, as well as a (new) radio button group:
o Image
o Text
o Other

"Image" attachments would be shown inline, as you do right now with screenshots (and people 
can comment on them by selecting rectangles).

"Text" attachements would be shown inline, very similar to the right side of a diff when you add a 
new file as part of your change. People would then be able to comment on your attachment just 
like they comment on the main change.

Finally, "Other" attachments would not be shown inline. But reviewers would have the ability to 
download them.

To prevent out-of-storage denial-of-service, a max size either per file upload, or per user.
chipx86
#1 chipx86
  • -Type-Defect
    +Type-Enhancement
    +Component-Reviews
chipx86
#2 chipx86
  • +Milestone-Release2.0
david
#3 david
  • +Confirmed
chipx86
#7 chipx86
  • -Milestone-Release2.0
    +Milestone-Release1.5
chipx86
#9 chipx86
  • +Milestone-Release1.6
chipx86
#10 chipx86
  • +Priority-High
chipx86
#11 chipx86
  • -Confirmed
    +Started
chipx86
#12 chipx86
Pushing out to 1.7 (tentatively).
  • +Milestone-Release1.7
#13 da.****@gmai***** (Google Code) (Is this you? Claim this profile.)
It'd be great for Reviewboard to be able to take a good guess at what type of file is being uploaded. 
chipx86
#14 chipx86
Added support for file attachments to master (34e752a). Some work needs to be done still, but this will be in 1.6.
  • -Started
    +Fixed
  • -Milestone-Release1.7
    +Milestone-Release1.6
  • +chipx86