Homec4science

File - add transactions and editor

Authored by Bob Trahan <bob.trahan@gmail.com> on Sep 5 2013, 22:11.

Description

File - add transactions and editor

Summary: this ends up being a little weird since you can't actually edit files. Also, since we create files all sorts of ways, sometimes without even having a user, we don't bother logging transactions for those events. Fixes T3651. Turns out this work is important for T3612, which is a priority of mine to help get Pholio out the door.

Test Plan: left a comment on a file. it worked! use bin/mail to verify mail content looked correct.

Reviewers: epriestley

Reviewed By: epriestley

CC: Korvin, aran, wez

Maniphest Tasks: T3651, T3612

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

Details

Committed
Bob Trahan <bob.trahan@gmail.com>Sep 5 2013, 22:11
Pushed
aubortJan 31 2017, 17:16
Parents
rPH4b061a766a9c: More Diviner style updates
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <bob.trahan@gmail.com> committed rPH228496cdbe16: File - add transactions and editor (authored by Bob Trahan <bob.trahan@gmail.com>).Sep 5 2013, 22:11