Skip to content

fixing outdated property names - #71

Open
moda20 wants to merge 1 commit into
flexxnn:masterfrom
moda20:master
Open

fixing outdated property names#71
moda20 wants to merge 1 commit into
flexxnn:masterfrom
moda20:master

Conversation

@moda20

@moda20 moda20 commented Jan 5, 2020

Copy link
Copy Markdown

changed the sequelize attribute to database and the attributes attribute on models to tableattributes

@moda20

moda20 commented Jan 5, 2020

Copy link
Copy Markdown
Author

This will create the right migrations with table attributes not only the table names
fixes : #69

@Naddyson

Copy link
Copy Markdown

This pull request has an issue. sequelize is not defined. I changed and it's working fine with autogenerated models/index.js

Row 72 makemigrations.js
From
let sequelize = require(modelsDir).database;
To
let sequelize = require(modelsDir);

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants