Db/seeding #296
Merged
Db/seeding #296
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
DVidal1205
previously requested changes
Jan 19, 2026
Contributor
DVidal1205
left a comment
DVidal1205
left a comment
There was a problem hiding this comment.
great work, just can never fully trust Discord
Contributor
|
@DVidal1205 u should review this 💯 |
hector1128
previously requested changes
Jan 20, 2026
Contributor
hector1128
left a comment
hector1128
left a comment
There was a problem hiding this comment.
Just a few tweaks but overall very solid. Good job Dhruv
farisAwheel
reviewed
Jan 21, 2026
Contributor
farisAwheel
left a comment
farisAwheel
left a comment
There was a problem hiding this comment.
ima give it the LGTM! i was about to say wrap the discord API calls w/ try catch but dylan beat me to it
went on a wild goose chase with ensuring that cleanTable actually gets everything, there are some tables that should be cleared but don't have a direct FK w/ the user table, just have an indirect link, but from what i checked, all of affected tables have a link to member which cascades on delete, so it should be fine
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why
Getting good data into our DB to be used for deving has been a pain forever so we needed a seeding script.
What
This introduces a seeding script and cron job that backs up the prod db (minus all sensitive information) and makes a backup sql file to insert those rows into our local db and saves it on our minio client. The seeding script can then pull this sql script and then fill the local rows with prod data that is fine for the dev team to have.
Test Plan
Idk man there's too many moving parts to take a video or screenshot just ask me lowk 🙏 Ofc feel free to lmk if the usage is too confusing I'll improve the docs and stuff