Home GnuPG

Merge multiple Auditors transactions from Herald
3b6a651b69feUnpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Merge multiple Auditors transactions from Herald

Summary:
Fixes T12302. Currently, we aren't merging multiple "AddAuditors" transactions correctly.

This can occur when Herald triggers multiple auditor rules.

Instead, merge them.

Test Plan:

  • Wrote two different Herald rules that add auditors.
  • Pushed a commit which triggered them.
  • After the change, saw all the auditors get added correctly.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12302

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

Details

Provenance
epriestley <git@epriestley.com>Authored on Feb 23 2017, 11:42 PM
Parents
rPHAB3eae9a368de0: Modular Transactions for Badges
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHAB3b6a651b69fe: Merge multiple Auditors transactions from Herald (authored by epriestley <git@epriestley.com>).Feb 24 2017, 12:14 AM