Homec4science

Migrate audit comments to transactions

Authored by epriestley <git@epriestley.com> on Jul 29 2014, 00:00.

Description

Migrate audit comments to transactions

Summary:
Ref T4896. Depends on D10052. This is the major/scary migration, but not really so bad. It is substantially similar to D8210, but less complex because there are fewer actions here.

This moves PhabricatorAuditComment storage to PhabricatorAuditTransaction, then reads PhabricatorAuditComments as a proxy around the new objects.

Test Plan:

  • Before migrating, browsed around. Nothing appeared broken.
  • Migrated cleanly.
  • Viewed old transactions (inlines, comments, accept/reject/etc, add auditors, add ccs, implicit CCs).
  • Added all of those comment types.
  • Edited a draft.
  • Deleted a draft.
  • Spot checked the database for sanity.

Reviewers: btrahan, joshuaspence

Reviewed By: joshuaspence

Subscribers: epriestley

Maniphest Tasks: T4896

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

Details

Committed
epriestley <git@epriestley.com>Jul 29 2014, 00:00
Pushed
aubortJan 31 2017, 17:16
Parents
rPH608e1d20b48a: Write separate comments for every action in Audit
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHf965126dc454: Migrate audit comments to transactions (authored by epriestley <git@epriestley.com>).Jul 29 2014, 00:00