Skip to content
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

Tech Debt: Update Sequelize & Polish + Harden DB Schema #172

Open
billdybas opened this issue Feb 15, 2018 · 1 comment
Open

Tech Debt: Update Sequelize & Polish + Harden DB Schema #172

billdybas opened this issue Feb 15, 2018 · 1 comment
Labels
Backend Task is Primarily Backend Tech Debt

Comments

@billdybas
Copy link
Member

We're using Sequelize v4, but I think some of the ways we do things are deprecated since v3 or have different syntax. Make sure we're not using outdated conventions and update if we are (http://docs.sequelizejs.com/manual/tutorial/upgrade-to-v4.html). And update our version to the latest v4.

Make sure that on tables we've already created, we have the correct types, default values, null constraints, validations.

Make sure that on relations we've already created, we have the correct on update and on delete actions.

We probably could squash our migrations down, too, since we're still pre-deploy.

@billdybas billdybas added Backend Task is Primarily Backend Tech Debt labels Mar 12, 2018
@billdybas
Copy link
Member Author

user_shows needs cascade deletes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend Task is Primarily Backend Tech Debt
Projects
None yet
Development

No branches or pull requests

1 participant