Show open issues instead of Ship It! in the dashboard and reviews.

Review Request #5286 — Created Jan. 20, 2014 and submitted — Latest diff uploaded

Information

Review Board
master

Reviewers

Show open issues instead of Ship It! in the dashboard and reviews.

The dashboard now shows the number of open issues instead of the number
of Ship Its if there are open issues. If there are no open issues left,
the Ship Its are displayed instead. This makes it a lot easier to see at
a glance on the dashboard if a change is ready to go in, or if work
still needs to be done.

On a review box, "Ship it!" will be replaced by "Fix it, then Ship it!"
when there are open issues. As soon as they're all resolved for that
review, the text will be replaced by "Ship it!" Likewise, re-opening an
issue will flip the text back to "Fix it, then Ship It!"

Tested the dashboard with review requests containing Ship Its, those containing
just issues, and those with a mix.

Tested loading review boxes with Ship Its, with issues, and without either.

Tested changing issue states to clear all open issues and to reopen, and saw
that the Ship It tag changed.


    Loading...