Homec4science

Migrate Audit inline comments to new storage

Authored by epriestley <git@epriestley.com> on Jul 25 2014, 02:59.

Description

Migrate Audit inline comments to new storage

Summary:
Ref T4896. This is substantially identical to the process which Differential followed, and mostly copied from the original Differential migration and the Differential proxy object.

Basically, we move all the data over but the application can't tell, and the same APIs do reads and writes to the new table.

Test Plan:

  • Browsed UI before migrating, everything looked fine (but no inlines).
  • Ran migration.
  • Verified draft and published comments survived migration.
  • Added a draft.
  • Previewed draft.
  • Submitted draft.
  • Viewed standalone with drafts and published comments.
  • Sanity checked data in database, didn't see anything unusual.

Reviewers: btrahan, joshuaspence

Reviewed By: joshuaspence

Subscribers: epriestley

Maniphest Tasks: T4896

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

Details

Committed
epriestley <git@epriestley.com>Jul 25 2014, 02:59
Pushed
aubortJan 31 2017, 17:16
Parents
rPH416f3d9ede5a: Add storage for new audit transactions and comments
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHc01aa794c1f2: Migrate Audit inline comments to new storage (authored by epriestley <git@epriestley.com>).Jul 25 2014, 02:59