rietveld-codereview / rietveld

Code Review, hosted on Google App Engine
https://codereview.appspot.com
Apache License 2.0
557 stars 152 forks source link

Notify removed reviewers on the next message sent on an issue #543

Open zetafunction opened 8 years ago

zetafunction commented 8 years ago

Sometimes, an issue owner needs to change reviewers. Usually, that happens by just editing the reviewers field and then sending out an email.

However, this doesn't notify the removed reviewer and they might not notice they don't need to review it anymore. Can we do something like what Monorail does: when you're removed, you get a notification about it. For Rietveld, we could do something simple: I think we could just include all reviewers that received mail in the last sent message when sending a new message.