Skip to content

ref(seer-settings): Remove unused helper to bulk write Seer project preferences - #128708

Open
srest2021 wants to merge 4 commits into
masterfrom
ref/seer-remove-bulk-write-preferences
Open

srest2021 wants to merge 4 commits into
masterfrom
ref/seer-remove-bulk-write-preferences

Conversation

@srest2021

@srest2021 srest2021 commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

Removes bulk_write_preferences_to_sentry_db and its tests. Its last caller went away with the organization autofix automation settings endpoint (#128533), and nothing calls it anymore.

With that gone, write_preference_to_sentry_db (called by the legacy ProjectSeerPreferencesEndpoint) was the only caller of _write_preferences_to_sentry_db, so the helper is inlined into it and simplified to a single project. No behavior change.

Relates to CW-2249

Its last caller was removed with the organization autofix automation settings endpoint.
@linear-code

linear-code Bot commented Oct 23, 2026

Copy link
Copy Markdown

CW-2249

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 23, 2026
@srest2021 srest2021 changed the title ref(seer): Remove unused bulk_write_preferences_to_sentry_db ref(seer-settings): Remove unused bulk_write_preferences_to_sentry_db helper Oct 23, 2026
@srest2021 srest2021 changed the title ref(seer-settings): Remove unused bulk_write_preferences_to_sentry_db helper ref(seer-settings): Remove unused helper to bulk write Seer project preferences Oct 23, 2026
@srest2021
srest2021 marked this pull request as ready for review October 23, 2026 22:14
@srest2021
srest2021 requested a review from a team as a code owner October 23, 2026 22:14
@srest2021
srest2021 requested a review from a team October 23, 2026 22:14

This branch was successfully deployed

1 active deployment
Preview — abe652c0 Deployed Oct 23, 2026 by vercel[bot]
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