Fix migrations - Githubissues

slackapi / node-tasks-app

Tasks App is a sample Task Management app built on the Slack Platform.

MIT License

99 stars 50 forks source link

Fix migrations #69

Closed colmdoyle closed 3 years ago

colmdoyle commented 3 years ago

Summary

In 3cea85ae600bdad6152ffd4b429a9d6d70ac2765, I inadvertently introduced a bug where migrations wouldn't run on some DB types, includingMySQL.

This was because I was mixing and matching DataTypes.

Normally editing existing migrations is a bad idea, but I'm going to do it here because

It's early in the project
Migrations won't run, so even if we fixed it with a migration, it wouldn't get to it.

closes #67

Requirements (place an `x` in each `[ ]`)

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.