-
Notifications
You must be signed in to change notification settings - Fork 405
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
improve deployment stage time execution + prevent loosing data in prod #141
Conversation
8483e20
to
ec65d65
Compare
Impressive one for improvement of deployment 🚀 Prevent database dropping might cause issues during development/deployment: SQL queries should be constructed and executed very rigorously from schema.sql with |
I agree with you. Managing migrations is complicated, and I think it's necessary for learners to be aware of the rigor these files require. It's good practice for their professional careers. Moreover, my learners have worked with this constraint and have not been hindered. Of course, they still have the option to delete the database if necessary, and the instructor can also do it in production via phpMyAdmin. Other solutions are also conceivable, such as using an ORM like Sequelize, which allows for better migration management. |
This pull request improves the deployment steps.