Fixes T12302. Currently, we aren't merging multiple "AddAuditors" transactions correctly.
This can occur when Herald triggers multiple auditor rules.
Instead, merge them.
Details
Details
- Reviewers
chad - Maniphest Tasks
- T12302: Herald adding different auditors but only the last one is kept
- Commits
- rP3b6a651b69fe: Merge multiple Auditors transactions from Herald
- Wrote two different Herald rules that add auditors.
- Pushed a commit which triggered them.
- After the change, saw all the auditors get added correctly.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- audit1
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 15746 Build 20809: Run Core Tests Build 20808: arc lint + arc unit