Skip to content

Conversation

@ceorourke
Copy link
Member

@ceorourke ceorourke commented Dec 3, 2024

When removing a model that has a hybrid cloud foreign key you can't set db_constraint to False, but instead you need to make the column nullable.

IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs to go live.

  • Urgent deadline (GA date, etc.):
  • Other deadline:
  • None: Not urgent, can wait up to 1 week+

SLA

  • Teamwork makes the dream work, so please add a reviewer to your PRs.
  • Please give the docs team up to 1 week to review your PR unless you've added an urgent due date to it.
    Thanks in advance for your help!

PRE-MERGE CHECKLIST

Make sure you've checked the following before merging your changes:

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

@vercel
Copy link

vercel bot commented Dec 3, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
develop-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 3, 2024 11:13pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
changelog ⬜️ Ignored (Inspect) Dec 3, 2024 11:13pm
sentry-docs ⬜️ Ignored (Inspect) Dec 3, 2024 11:13pm

@ceorourke ceorourke merged commit 7841916 into master Dec 4, 2024
10 checks passed
@ceorourke ceorourke deleted the ceorourke/hcfk-migration branch December 4, 2024 17:33
@coolguyzone coolguyzone self-requested a review December 4, 2024 21:23
@github-actions github-actions bot locked and limited conversation to collaborators Dec 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants