Skip to content

chore(code-review): Remove the deprecated prevent config v0#106993

Merged
suejung-sentry merged 5 commits intomasterfrom
sshin/delete-prevent-config
Jan 27, 2026
Merged

chore(code-review): Remove the deprecated prevent config v0#106993
suejung-sentry merged 5 commits intomasterfrom
sshin/delete-prevent-config

Conversation

@suejung-sentry
Copy link
Contributor

@suejung-sentry suejung-sentry commented Jan 26, 2026

This v0 of the "prevent" configuration model is superseded by all the new stuff for code review GA that now lives in repo settings table.
Delete these apis and in a subsequent PR delete the whole table too.

Follows instructions here for table deletion

All cleanup PRs here:

  1. Remove all references to the table - link - THIS PR
  2. Move the table to Pending Delete - link
  3. Move the table to Delete - link
  4. Remove the "prevent" app - link

Relates to CW-684

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jan 26, 2026
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 26, 2026
@github-actions
Copy link
Contributor

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

Model for managing configurations for Prevent AI features
per Sentry organization and integration.

DEPRECATED: This model is no longer used and will be removed in a future PR.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

See these PRs for table removal:
#107003
#107013

@linear
Copy link

linear bot commented Jan 26, 2026

@suejung-sentry suejung-sentry marked this pull request as ready for review January 26, 2026 23:44
@suejung-sentry suejung-sentry requested review from a team as code owners January 26, 2026 23:44
Copy link
Contributor

@ajay-sentry ajay-sentry left a comment

Choose a reason for hiding this comment

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

yay

@suejung-sentry suejung-sentry merged commit 34bbe8b into master Jan 27, 2026
69 checks passed
@suejung-sentry suejung-sentry deleted the sshin/delete-prevent-config branch January 27, 2026 19:23
suejung-sentry pushed a commit that referenced this pull request Jan 27, 2026
…107003)

We don't need the `prevent_ai_configuration` table anymore, nor the
whole app for "prevent". This is step 2 of 4

All cleanup PRs here:
1. Remove all references to the table -
[link](#106993)
2. Move the table to Pending Delete -
[link](https://github.com/getsentry/sentry/pull/107003/files) - THIS PR
3. Move the table to Delete -
[link](https://github.com/getsentry/sentry/pull/107013/files)
4. Remove the "prevent" app -
[link](https://github.com/getsentry/sentry/pull/107015/files)

Relates to
[CW-684](https://linear.app/getsentry/issue/CW-684/delete-old-prevent-configuration-table-and-related-code)

---------

Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>
priscilawebdev pushed a commit that referenced this pull request Jan 28, 2026
…107003)

We don't need the `prevent_ai_configuration` table anymore, nor the
whole app for "prevent". This is step 2 of 4

All cleanup PRs here:
1. Remove all references to the table -
[link](#106993)
2. Move the table to Pending Delete -
[link](https://github.com/getsentry/sentry/pull/107003/files) - THIS PR
3. Move the table to Delete -
[link](https://github.com/getsentry/sentry/pull/107013/files)
4. Remove the "prevent" app -
[link](https://github.com/getsentry/sentry/pull/107015/files)

Relates to
[CW-684](https://linear.app/getsentry/issue/CW-684/delete-old-prevent-configuration-table-and-related-code)

---------

Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>
suejung-sentry pushed a commit that referenced this pull request Jan 29, 2026
We don't need the prevent_ai_configuration table anymore, nor the whole
app for "prevent". This is step 3 of 4

All cleanup PRs here:
1. Remove all references to the table -
[link](#106993)
2. Move the table to Pending Delete -
[link](https://github.com/getsentry/sentry/pull/107003/files)
3. Move the table to Delete -
[link](https://github.com/getsentry/sentry/pull/107013/files) - THIS PR
4. Remove the "prevent" app -
[link](https://github.com/getsentry/sentry/pull/107015/files)

Closes
[CW-684](https://linear.app/getsentry/issue/CW-684/delete-old-prevent-configuration-table-and-related-code)

---------

Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>
@suejung-sentry suejung-sentry self-assigned this Jan 29, 2026
priscilawebdev pushed a commit that referenced this pull request Feb 2, 2026
This v0 of the "prevent" configuration model is superseded by all the
new stuff for code review GA that now lives in repo settings table.
Delete these apis and in a subsequent PR delete the whole table too.

Follows instructions
[here](https://develop.sentry.dev/backend/application-domains/database-migrations/#deleting-tables)
for table deletion

All cleanup PRs here:
1. Remove all references to the table -
[link](#106993) - THIS PR
2. Move the table to Pending Delete -
[link](https://github.com/getsentry/sentry/pull/107003/files)
3. Move the table to Delete -
[link](https://github.com/getsentry/sentry/pull/107013/files)
4. Remove the "prevent" app -
[link](https://github.com/getsentry/sentry/pull/107015/files)

Relates to
[CW-684](https://linear.app/getsentry/issue/CW-684/delete-old-prevent-configuration-table-and-related-code)

---------

Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>
priscilawebdev pushed a commit that referenced this pull request Feb 2, 2026
…107003)

We don't need the `prevent_ai_configuration` table anymore, nor the
whole app for "prevent". This is step 2 of 4

All cleanup PRs here:
1. Remove all references to the table -
[link](#106993)
2. Move the table to Pending Delete -
[link](https://github.com/getsentry/sentry/pull/107003/files) - THIS PR
3. Move the table to Delete -
[link](https://github.com/getsentry/sentry/pull/107013/files)
4. Remove the "prevent" app -
[link](https://github.com/getsentry/sentry/pull/107015/files)

Relates to
[CW-684](https://linear.app/getsentry/issue/CW-684/delete-old-prevent-configuration-table-and-related-code)

---------

Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>
priscilawebdev pushed a commit that referenced this pull request Feb 2, 2026
We don't need the prevent_ai_configuration table anymore, nor the whole
app for "prevent". This is step 3 of 4

All cleanup PRs here:
1. Remove all references to the table -
[link](#106993)
2. Move the table to Pending Delete -
[link](https://github.com/getsentry/sentry/pull/107003/files)
3. Move the table to Delete -
[link](https://github.com/getsentry/sentry/pull/107013/files) - THIS PR
4. Remove the "prevent" app -
[link](https://github.com/getsentry/sentry/pull/107015/files)

Closes
[CW-684](https://linear.app/getsentry/issue/CW-684/delete-old-prevent-configuration-table-and-related-code)

---------

Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>
@github-actions github-actions bot locked and limited conversation to collaborators Feb 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants