Skip to content

fix(budget): cascade-delete orphaned categories when deleting a budget - #377

Merged
arutsh merged 3 commits into
mainfrom
Budget/fix/Issue-346/category-cascade-delete-fix-group1
Oct 1, 2026
Merged

arutsh merged 3 commits into
mainfrom
Budget/fix/Issue-346/category-cascade-delete-fix-group1

Conversation

@arutsh

@arutsh arutsh commented Oct 1, 2026

Copy link
Copy Markdown
Owner

BudgetModel.categories now uses cascade="all, delete-orphan" with passive_deletes=True. Previously, deleting a budget whose category had outlived its lines nulled out budget_id and failed with an IntegrityError (400).

Adds real-DB tests for both the unloaded and the already-loaded categories cases, and updates the design/tasks for the change.

Refs #346

BudgetModel.categories now uses cascade="all, delete-orphan" with
passive_deletes=True. Previously, deleting a budget whose category had
outlived its lines nulled out budget_id and failed with an IntegrityError (400).

Adds real-DB tests for both the unloaded and the already-loaded
categories cases, and updates the design/tasks for the change.

Refs #346

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
grant-flow Ready Ready Preview Oct 1, 2026 3:16pm UTC

quay.io/minio/minio now requires auth and minio/minio is gone from Docker
Hub, so the E2E workflow failed at image pull on every branch. Switch the
local and dev stacks to the pinned community build of the same server, and
fold the swap into the 295 change's proposal, design and tasks.

Refs #346

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@arutsh
arutsh merged commit 58a650f into main Oct 1, 2026
27 of 29 checks passed
@arutsh
arutsh deleted the Budget/fix/Issue-346/category-cascade-delete-fix-group1 branch October 1, 2026 15:34

This branch was successfully deployed

1 active deployment
Preview — f5b68b89 Deployed Oct 1, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant