Skip to content

chore(searches): Remove user level saved search code - #118880

Merged
ceorourke merged 1 commit into
masterfrom
ceorourke/remove-user-level-saved-searches
Jul 7, 2026
Merged

chore(searches): Remove user level saved search code#118880
ceorourke merged 1 commit into
masterfrom
ceorourke/remove-user-level-saved-searches

Conversation

@ceorourke

Copy link
Copy Markdown
Member

These have long since been replaced by starred searches - this removes the last front end pieces of it so that we can remove the endpoints and then the SavedSearch model.

@ceorourke
ceorourke requested a review from a team as a code owner July 1, 2026 22:03
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jul 1, 2026
@ceorourke
ceorourke requested a review from malwilley July 7, 2026 17:37

@malwilley malwilley left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

@ceorourke
ceorourke merged commit 1aaa727 into master Jul 7, 2026
73 checks passed
@ceorourke
ceorourke deleted the ceorourke/remove-user-level-saved-searches branch July 7, 2026 17:42
ceorourke added a commit that referenced this pull request Jul 7, 2026
Remove the unused saved search backend. None of the endpoints were
public and they are barely used anyway (~60 API calls in the last 2 days
for the `OrganizationSearchesEndpoint`, 0 for the others). Updates some
tests to use `Dashboard` instead, since the tests were more about
cascade deletion rather than the searches.

Do not merge until after #118880

---------

Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants