Page MenuHomePhabricator

Make "EditPro" controller work with diff updates
ClosedPublic

Authored by epriestley on Feb 28 2014, 11:12 PM.
Tags
None
Referenced Files
F14082809: D8376.diff
Sat, Nov 23, 1:25 AM
Unknown Object (File)
Wed, Nov 20, 11:12 PM
Unknown Object (File)
Sun, Nov 17, 1:55 AM
Unknown Object (File)
Sun, Oct 27, 11:06 PM
Unknown Object (File)
Sun, Oct 27, 12:17 AM
Unknown Object (File)
Oct 17 2024, 5:55 PM
Unknown Object (File)
Oct 14 2024, 11:03 PM
Unknown Object (File)
Oct 12 2024, 7:40 AM
Subscribers

Details

Summary

Ref T2222. Make the "EditPro" controller accommodate diff updates, and support the transaction type. This one is pretty straightforward.

Also make revisionPHID in the comments table nullable to fix the "Edit" action.

Test Plan
  • Created new revision.
  • Updated revision.
  • Tried to do some invalid stuff.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Unknown Object (????).Feb 28 2014, 11:13 PM
  • Hold the /edit/ -> /editpro/ change.
epriestley updated this revision to Unknown Object (????).Feb 28 2014, 11:14 PM
  • Remove unused property.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 1 2014, 12:30 AM