Homec4science

Move audit to application transactions

Authored by Bob Trahan <btrahan@phacility.com> on Oct 15 2014, 22:20.

Description

Move audit to application transactions

Summary:
Fixes T4896, T6293.

Do most of the work in the editor, but pull the raw patch in the daemon and set that on the editor. This is somewhat of a pre-optimization but it was easy enough to do and makes sense to me.

Test Plan:
made a commit and saw it get parsed.
made a commit with "Auditors: foo" field and saw audit made for foo
turned on inline patch and attach patch and saw the patches

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin, epriestley

Maniphest Tasks: T6293, T4896

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

Details

Committed
Bob Trahan <btrahan@phacility.com>Oct 15 2014, 22:20
Pushed
aubortJan 31 2017, 17:16
Parents
rPH87679d4295ea: Add actions to crumbs in fund/phortune
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <btrahan@phacility.com> committed rPH3dd92a78ac2a: Move audit to application transactions (authored by Bob Trahan <btrahan@phacility.com>).Oct 15 2014, 22:20