-
Notifications
You must be signed in to change notification settings - Fork 2.1k
feat(surveys): add FK for linked_insight_id #41708
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Migration SQL ChangesHey 👋, we've detected some migrations on this PR. Here's the SQL output for each migration, make sure they make sense:
|
🔍 Migration Risk AnalysisWe've analyzed your migrations for potential risks. Summary: 1 Safe | 0 Needs Review | 0 Blocked ✅ SafeNo contention risk, backwards compatible Last updated: 2025-11-19 23:33 UTC (b54e6e3) |
71dabc7 to
755e84f
Compare
5298d5d to
05ba8ee
Compare
d2d3fa8 to
d402eb6
Compare
d402eb6 to
84e0972
Compare
38f2bb7 to
258102b
Compare
c934b85 to
f69a7a5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
5 files reviewed, no comments
Edit Code Review Agent Settings | Greptile
React with 👍 or 👎 to share your feedback on this new summary format
adamleithp
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Low context stamp
f69a7a5 to
23c72bf
Compare

Problem
step 2 of 4 for this migration....
this was previously approved as one big PR (#41574) but i had to split it up due to restrictive migration CI checks :(
this will also likely need a rebase, but just want to get approvals so i can pop it in the queue when ready
Changes
How did you test this code?
👉 Stay up-to-date with PostHog coding conventions for a smoother review.
Changelog: (features only) Is this feature complete?