Home GnuPG

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

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

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
rPHAB163e1ec4426e: Expose the commit/task/revision relationship edges to "edge.search"
Branches
Unknown
Tags
Unknown

Event Timeline

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