Hi James,

I think this is an interesting idea. We'd need to talk about it more, see
how it'd fit into things, but it might be worth putting together. Either
natively or as an extension.

A quicker option for now is to make use of the Note To Reviewers extension,
which gives you a text field below Testing Done that you can populate with
whatever information you want to communicate to reviewers. The advantage of
this is that the history of notes is recorded, just like any other field,
so people can see when the change was "suspended" and when it was made
available again for review.

Christian

On Tue, Aug 28, 2018 at 7:45 AM James Carlson <[email protected]>
wrote:

> Every now and then, I need to tell my reviewers that there's a serious
> issue that makes their current review efforts moot.  Sometimes it's a large
> change in the source repository, and I'd like to merge (and possibly
> redesign) before continuing.  Sometimes it's due to someone's comment that
> sends me in a different direction.  Sometimes it's due to late test results
> that show an issue that wasn't caught by the initial tests.
>
> Whatever the cause, I'd like to be able to "suspend" a review.  I don't
> want to lose the existing review discussion, and I don't want to tell
> reviewers that I'm just going away, so Close->Discarded doesn't feel
> right.  And there's no good way to just add a comment to the top of the
> review.  Usually, I end up modifying the review "Description" or "Testing
> Done" field to indicate what's going on, but that seems less than adequate.
>
> Could we please have an "Update->Suspend" or something like that?  The
> work flow would be relatively simple: the request stays suspended until a
> new diff or file is added, and then it goes back to normal.
>
> --
> Supercharge your Review Board with Power Pack:
> https://www.reviewboard.org/powerpack/
> Want us to host Review Board for you? Check out RBCommons:
> https://rbcommons.com/
> Happy user? Let us know! https://www.reviewboard.org/users/
> ---
> You received this message because you are subscribed to the Google Groups
> "Review Board Community" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to [email protected].
> For more options, visit https://groups.google.com/d/optout.
>


-- 
Christian Hammond
President/CEO of Beanbag <https://www.beanbaginc.com/>
Makers of Review Board <https://www.reviewboard.org/>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"Review Board Community" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to