search

aljorhythm / sql-to-laravel-migrations

Python script that converts sql database tables into laravel migrations
113 stars 47 forks source link

php script doesn't create foreign key restraints #4

Open jymden opened 6 years ago

jymden commented 6 years ago

php script doesn't seem to create foreign keys in the generated migrations

shahsani commented 6 years ago

Any fresh update on foreign key constraints, please?

aljorhythm commented 6 years ago

@shahsani you are more than welcomed to contribute, unfortunately I am currently not working on any Laravel project so I will not be devoting time to this

shahsani commented 6 years ago

Kindly tell us how did you change the engine to InnoDB. I mean where in code you did the changes!

Thanks

RAEES

From: Christoffer Wadensten notifications@github.com Sent: Friday, January 12, 2018 5:02 PM To: aljorhythm/sql-to-laravel-migrations Cc: Raees Khan ShahSani; Comment Subject: Re: [aljorhythm/sql-to-laravel-migrations] php script doesn't create foreign key restraints (#4)

I solved this by changing default engine to innodb. Works well with that. Don't understand why Laravel uses myisam

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/aljorhythm/sql-to-laravel-migrations/issues/4#issuecomment-357221438, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AC-Mmgym5Jw5CMYjY5evgpDjmeFfHAS8ks5tJ0ndgaJpZM4QjYW7.

shahsani commented 6 years ago

@aljorhythm indeed I want to contribute. I will, after going through the code! :-)