• 
      

    Add parent diff support to the pre-commit section of "New Review Request".

    Review Request #6117 — Created July 18, 2014 and submitted — Latest diff uploaded

    Information

    Review Board
    release-2.0.x
    ae22705...

    Reviewers

    In general, the new "New Review Request" page works very well for people. One
    feature that was missed by a small number of users was the ability to specify a
    parent diff. This change adds back that feature.

    If the given diff fails due to missing files or revisions, the user is shown
    the error, but additionally given the option of uploading a parent diff. This
    keeps the streamlined workflow for the general case but makes it possible for
    the complex one.

    While I was in here, I also plumbed through errors when creating the review
    request or diff to display. I don't think we've ever hit these errors in
    practice, due to the diff validation step (I had to artificially hack the code
    to trigger them).

    • Uploaded a plain diff via both DnD and the select button and saw it create
      the review request.
    • Uploaded a diff which required a parent diff and saw the expected error
      message with new form.
    • Uploaded diff + parent diff using both DnD and the select button, and saw it
      create the review request.
    • Uploaded a broken diff and saw the right error.
    • Uploaded a diff and parent diff which still required another parent to
      completely resolve all changes and saw an error.
    • Checked that "Start over" worked on both error pages.
    • Ran jshint
    • Ran js-tests