search

cviebrock / sequel-pro-laravel-export

A Sequel Pro / Sequel Ace bundle to generate Laravel migration files from existing tables.
MIT License
921 stars 55 forks source link

JSON field type not mapped correctly #7

Closed tomcoonen closed 7 years ago

tomcoonen commented 7 years ago

Field creation syntax:

`some_json_field` json DEFAULT NULL

Creates this migration:

 $table->UNKNOWN:json('some_json_field')->nullable();

Which should be:

$table->json('some_json_field')->nullable();
cviebrock commented 7 years ago

Thanks Tom. None of the instances of MySQL I have handy support JSON columns, so would you mind testing the master version of the package and see if the change I added fixes it?

Thanks for the testing!

tomcoonen commented 7 years ago

Fixed!

cviebrock commented 7 years ago

Version 1.0-beta.3 has this and the increments bug (#6) fix, plus a fix for ENUMs that I found. Thanks again!