-
Notifications
You must be signed in to change notification settings - Fork 0
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
Generate Seed Data #161
Generate Seed Data #161
Conversation
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.
Other than the issue pointed out earlier, this is good!
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.
Didn't test yet but LGTM! 🚢
What do we think about adding a prisma db clear
script as well to delete all data (but not drop tables)? prisma/docs#451 (comment)
seems pretty low priority. Is the advantage just making the data reset faster since you don't have to wait for migrations to run? Bc prisma migrate reset clears all the data and reruns all migrations. |
Ticket link
Closes #92
Steps to test
Ask me for the UIDs of the test users to add to your .env.
run
docker exec -it sistering_backend bash -c "npx prisma db seed"
to run just the seed script ordocker exec -it sistering_backend bash -c "npx prisma migrate reset"
to replace data in your current db with seed data.Try logging in as a test user. The password for all test users is Test123.
What should reviewers focus on?
Checklist