Skip to content

Conversation

jerryzhou196
Copy link
Contributor

@jerryzhou196 jerryzhou196 commented Sep 12, 2025

This is Stage 1 for removing the Feedback model.

Sets the Feedback FK to not be a db constraint as per the Develop Migrations Docs

@jerryzhou196 jerryzhou196 requested review from a team as code owners September 12, 2025 15:48
Copy link

linear bot commented Sep 12, 2025

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Sep 12, 2025
Copy link
Contributor

github-actions bot commented Sep 12, 2025

This PR has a migration; here is the generated SQL for src/sentry/feedback/migrations/0005_feedback_fk_not_db_contstr.py

for 0005_feedback_fk_not_db_contstr in feedback

--
-- Alter field environment on feedback
--
SET CONSTRAINTS "feedback_feedback_environment_id_fab8783f_fk_sentry_en" IMMEDIATE; ALTER TABLE "feedback_feedback" DROP CONSTRAINT "feedback_feedback_environment_id_fab8783f_fk_sentry_en";
--
-- Moved model feedback to pending deletion state
--
-- (no-op)

cursor[bot]

This comment was marked as outdated.

@jerryzhou196 jerryzhou196 enabled auto-merge (squash) September 12, 2025 16:40
Copy link

codecov bot commented Sep 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #99379      +/-   ##
==========================================
- Coverage   81.19%   81.19%   -0.01%     
==========================================
  Files        8591     8590       -1     
  Lines      380188   380142      -46     
  Branches    24122    24115       -7     
==========================================
- Hits       308703   308660      -43     
+ Misses      71131    71129       -2     
+ Partials      354      353       -1     

@jerryzhou196 jerryzhou196 merged commit 306195a into master Sep 12, 2025
64 of 65 checks passed
@jerryzhou196 jerryzhou196 deleted the REPLAY-610-clean-up-postgres-model-1 branch September 12, 2025 17:02
jerryzhou196 added a commit that referenced this pull request Sep 15, 2025
This is Stage 2 for removing the `Feedback` Model.
 
Follow-up to #99379 that removes the `environment` FK field as per our
[Develop Migration
Docs](https://develop.sentry.dev/backend/application-domains/database-migrations/#deleting-tables)
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.

2 participants