Customers.notify_me can be a boolean

FACG4 / nigerian-airlines

MIT License

0 stars 1 forks source link

Customers.notify_me can be a boolean #15

Open tomduggan85 opened 6 years ago

tomduggan85 commented 6 years ago

https://github.com/FACG4/nigerian-airlines/blob/bb60785ca1a4ea3d7b8a50f0d9ef8c643df6965f/server/database/models/customers.js#L20

Instead of using a STRING as the datatype of notify_me, you could use a boolean (DataTypes.BOOLEAN), as the only values you'd expect to store there is true or false.

amusameh commented 6 years ago

Yeah it has to be boolean, fixed thanks