Ref T4896. Now that we have a transaction editor, we can delete a giant block of hacks.
I believe this also resolves the commit/task attachment issues @joshuaspence and @mbishopim3 mentioned.
Details
Details
- Reviewers
joshuaspence mbishopim3 btrahan - Maniphest Tasks
- T4896: Move Audit to ApplicationTransactions
- Commits
- Restricted Diffusion Commit
rPe8d272b0dad1: Use standard infrastructure to attach commits to other objects
Attached and detached commits and tasks.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
Comment Actions
Let me know if you still see issues with relationships created after this lands. (It won't fix prior relationships, but new ones should be fixed.)
Comment Actions
I can't strictly speak to everything moved in this diff, but applying relationships does work correctly now so it's functionally fine.