Review Board

beta

saving a draft via JSON unnecessarily requires a diff_revision field

Updated 10 months, 4 weeks ago

Michael Plump Reviewers
trunk reviewboard
None Review Board SVN 
I'm not sure what this diff_revision flag is used for.  You go ahead and fetch the diffset using it, but then you don't actually do anything with it.  I'm assuming it's not needed, but perhaps there's some subtelty I'm missing.
I've published reviews via JSON after taking this code out.
Ship it!
Christian Hammond
Posted 10 months, 4 weeks ago (February 12th, 2008, 1:53 p.m.)
Looks good! Committed to SVN.

The reason this was here is because reviews used to be tied to a diff. They're not anymore, but it looks like this code was left over.