What version are you running? V1.5.5 What's the URL of the page this enhancement relates to, if any? /dashboard/ Describe the enhancement and the motivation for it. Enhancement: Clear a review's "ship it" status in the dashboard when a new diff is added to a review after a reviewer has selected "ship it". Motivation: In our day to day use of Reviewboard our reviews go back & forth for quite a while and it is very common for one reviewer to say "ship it" and another to identify a problem afterwards. Revisions are made to the software and the diff updated (often many times) meaning that the "ship it" status on the dashboard very rarely indicates the actual current status (or acceptability) of the review. What operating system are you using? What browser? Win7, chrome Please provide any additional information below. N/A
We'll likely revisit how Ship It works in the next release. It certainly needs some changes made.
+ Milestone-Release1.7 + Component-Reviews
- Milestone-Release1.7 + Milestone-Release1.8
Adding my name to the list of people/companies who would love to see this functionality.
I added a litte patch as a work-around.
Adding my company to the list. This is definitely how we use ship its, and find the "ship it count" very confusing (but still critical... just really annoying 'cause it's both critical and wrong :D )
Would love to see this fixed. Five years after the original request.
I added our stuff to an extension that we use internally.
Maybe someone will use this, too.
Use "pip wheel ." to get an installable whl file.
This will add an additional column "Approval" and set the approval flag that can be checked with the WebAPI.
This is a really simple extension. Maybe we will add some configuration to it. Or someone wants to contribute. :-)
If you use Mercurial... you could use our hook, too.
Since last commit (https://github.com/reviewboard/reviewboard/commit/260637db51adf475d16a437cc9a38aa15d6b4720) the ShipIt column was improved for "issue-verifying".
There is an explicit "approved" flag of reviewboard but there is no indicator for this.
Every extension or API user should use "approved" to check if a review request is "approved".
But reviewboard itself uses the incorrect "ShipIt" count directly. Why not use another color for the shipit-count if "approved" is false? Like I did in a previous patch. I don't mean to use a calculation for another ShipIt count. Just to indicate the "approved" flag of reviewboard API.
Since 3.0 it is possible to revoke ShipIts. I added this to my extension. After someone posts an update it will revoke any previous ShipIt.