tree 261a7ee8e64dda1f6b9592a16c495270e3115bec
parent 0936e9bd1a118555a51f5e0da702cf6854a099b9
author Gal Paikin <paiking@google.com> 1618469914 +0200
committer Gal Paikin <paiking@google.com> 1618591333 +0200

Implement batch removal of reviewers in PostReview

Currently, it's fully supported to add multiple reviewers in PostReview,
and it's supported to delete reviewers via the delete reviewer endpoint
(only one per endpoint). However, it is not supported to delete
multiple reviewers in one endpoint in any way. This may cause issues
since currently to delete multiple reviewers the frontend calls the
delete reviewer endpoint many times.

This change extends PostReview to support deletion of reviewers on top
of additions. We ensure only one BatchUpdate is performed,
and only one email is sent.

It's unclear why we're sending the emails via NewChange.soy, but it is
out of scope for this change.

Change-Id: I088a8dc9bcf1390bc30e829ba33d1b52ce126c7b