Home GnuPG

(stable) Don't downgrade accepts on update (fix "sticky accept")
520d3b939204Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

(no title)

(stable) Don't downgrade accepts on update (fix "sticky accept")

Summary:
Fixes T12496. Sticky accept was accidentally impacted by the "void" changes in D17566.

Instead, don't always downgrade all accepts/rejects: on update, we only want to downgrade accepts.

Test Plan:

  • With sticky accept off, updated an accepted revision: new state is "needs review".
  • With sticky accept on, updated an accepted revision: new state is "accepted" (sticky accept working correctly).
  • Did "reject" + "request review" to make sure that still works, worked fine.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12496

Differential Revision: https://secure.phabricator.com/D17605

Details

Provenance
epriestley <git@epriestley.com>Authored on Apr 3 2017, 5:35 PM
Parents
rPHAB2460755603cb: (stable) Promote 2017 Week 13
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHAB520d3b939204: (stable) Don't downgrade accepts on update (fix "sticky accept") (authored by epriestley <git@epriestley.com>).Apr 3 2017, 6:55 PM