Status: Confirmed
        Labels: Component-Reviews

Comment #4 on issue 3329 by chip...@gmail.com: Show proper error message when attempting to publish review request with duplicate commit ID

This is certainly harder to hit now, but not fully fixed.

The following repro case results in two review requests with the same commit ID:

1. Run `rbt post`
2. Update the branch, run `rbt post` again, creating a new review request.
3. Update the original review request with `rbt post -r <ID>`.
4. Publish both review requests.

While it doesn't seem to break anymore (fortunately), it does result in two review requests with the same commit ID.

We need to check more thoroughly that publishing a draft wouldn't result in a duplicate commit ID, before any actual publishing takes place.

You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:

You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.

Reply via email to