Fixes T8237. This table is unusual and doesn't have an id column, so delete() doesn't actually know how to delete records and fails.
Details
Details
- Reviewers
btrahan - Maniphest Tasks
- T8237: Destroying (with ./bin/remove destroy CLI) an object (here a project not tested for other object) raise an error.
- Commits
- Restricted Diffusion Commit
rP4250719d1008: Destroy notifications properly from `bin/remove destroy`
- Used bin/remove destroy to destroy an object with notifications, as per T8237.
- Applied patch.
- Used bin/remove destroy to get a clean delete.
- Verified related notifications vanished from notification menu.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- destroynotif
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 6075 Build 6095: [Placeholder Plan] Wait for 30 Seconds