Skip to content

chore(savedsearch): Remove SavedSearch model#119156

Open
ceorourke wants to merge 2 commits into
masterfrom
ceorourke/rm-savedsearch
Open

chore(savedsearch): Remove SavedSearch model#119156
ceorourke wants to merge 2 commits into
masterfrom
ceorourke/rm-savedsearch

Conversation

@ceorourke

Copy link
Copy Markdown
Member

Follow up to #118883 to remove the SavedSearch model. A final follow up will do deletion_action=DeletionAction.DELETE to remove the table from Postgres.

@ceorourke ceorourke requested review from a team as code owners July 7, 2026 21:04
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 7, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit ddadad1. Configure here.

Comment thread src/sentry/models/savedsearch.py
Comment thread src/sentry/models/savedsearch.py
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

This PR has a migration; here is the generated SQL for src/sentry/migrations/1128_remove_savedsearch.py

for 1128_remove_savedsearch in sentry

--
-- Alter field organization on savedsearch
--
SET CONSTRAINTS "sentry_savedsearch_organization_id_3fd9781e_fk_sentry_or" IMMEDIATE; ALTER TABLE "sentry_savedsearch" DROP CONSTRAINT "sentry_savedsearch_organization_id_3fd9781e_fk_sentry_or";
--
-- Moved model SavedSearch to pending deletion state
--
-- (no-op)

@ceorourke ceorourke requested a review from a team July 7, 2026 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant