search

matthewtrask / elephpant-traders

Code for elephpanttrade.com
http://elephpanttrade.com/#/
6 stars 3 forks source link

When Creating User Locally, SQL Error 1364, Field 'image' doesn't have a default value... #26

Closed ephbaum closed 7 years ago

ephbaum commented 7 years ago

Just saw this attempting to register an account for myself locally.

Easy enough to fix to actually work on issue I wanted to do to contribute, but thought I'd write up an issue anyway.

[2017-08-29 21:57:42] local.ERROR: PDOException: SQLSTATE[HY000]: General error: 1364 Field 'image' doesn't have a default value in /home/vagrant/elephpant-traders/vendor/laravel/framework/src/Illuminate/Database/Connection.php:449
Stack trace: ...

Sure enough, create syntax for that table shows NOT NULL.

AMA
ephbaum commented 7 years ago

Interesting, looks like that should be handled by 4c76d592546f7a4b9cf4a6f2e89341ae61b4cd39 ?

matthewtrask commented 7 years ago

it needs to be set to null, which isn't in a migrant. One minute and Ill have it fixed :)

matthewtrask commented 7 years ago

fixed in 8c23fc8

ephbaum commented 7 years ago

Yeah, I fixed the locally. Wasn't sure about doing PR for it and then was digging into posts, discovering the lack of AWS credentials.

Was going to work on a couple partials for Facebook and Twitter meta tags. On Tue, Aug 29, 2017 at 6:18 PM Matt Trask notifications@github.com wrote:

it needs to be set to null, which isn't in a migrant. One minute and Ill have it fixed :)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/matthewtrask/elephpant-traders/issues/26#issuecomment-325831645, or mute the thread https://github.com/notifications/unsubscribe-auth/AD2b_FkB3lBy2lsiJPihQDsYja9hL9X1ks5sdJwqgaJpZM4PGnYg .

matthewtrask commented 7 years ago

the .env.example should have the fields for the AWS creds, which helped I hope.

ephbaum commented 7 years ago

Perfect, I'll roll back mine and delete, update and use yours. 👍 On Tue, Aug 29, 2017 at 6:28 PM F. Stephen Kirschbaum < fskirschbaum@gmail.com> wrote:

Yeah, I fixed the locally. Wasn't sure about doing PR for it and then was digging into posts, discovering the lack of AWS credentials.

Was going to work on a couple partials for Facebook and Twitter meta tags. On Tue, Aug 29, 2017 at 6:18 PM Matt Trask notifications@github.com wrote:

it needs to be set to null, which isn't in a migrant. One minute and Ill have it fixed :)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/matthewtrask/elephpant-traders/issues/26#issuecomment-325831645, or mute the thread https://github.com/notifications/unsubscribe-auth/AD2b_FkB3lBy2lsiJPihQDsYja9hL9X1ks5sdJwqgaJpZM4PGnYg .

matthewtrask commented 7 years ago

awesome, thanks for the feedback! Lemme know if you have any other issues!