HomePhorge

Destroy notifications properly from `bin/remove destroy`

Description

Destroy notifications properly from bin/remove destroy

Summary: 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.

Test Plan:

  • 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.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T8237

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

Details

Provenance
epriestleyAuthored on May 18 2015, 12:29 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP107d6db1e8f6: Don't put a spacer line on the bottom of previews
Branches
Unknown
Tags
Unknown

Event Timeline