Conversation
dessalines
commented
Apr 21, 2026
dessalines
commented
- This failed on lemmy.ml prod data
- Must have been copied from the wrong place.
- This failed on lemmy.ml prod data
|
Looks like you are basically reverting the change from #6360. I suppose you need to edit the migration and set these columns where they are missing. |
|
These are really difficult to discover where there's issues with prod data, so I'm going to first get through the migrations, then try to fix this constraint with a new migration. |
There was a problem hiding this comment.
Undoing the changes, here (IE using the current modlog checks), and adding a migration later to fix the broken rows.
|
Verified that this migration is working with prod data. |
| FROM | ||
| post p | ||
| WHERE | ||
| target_post_id = p.id; |
There was a problem hiding this comment.
Once its working, you can add these updates to the existing migration and get rid of this new migration.
There was a problem hiding this comment.
This one I'd prefer to keep as a separate migration, because
- It doesn't have to do with adding the modlog warnings, and
- because its easier to fix if its its own specific migration. It was really difficult to do this without going through every type of modlog action, so its nice to have an
up/downfor this one specifically.
2 participants
