HomePhorge

Merge multiple Auditors transactions from Herald

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
epriestleyAuthored on Feb 23 2017, 2:42 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP3eae9a368de0: Modular Transactions for Badges
Branches
Unknown
Tags
Unknown

Event Timeline