the generated config and the generated schemas had syntax issues or things missing

jhartwell / Plsm

Elixir mix task to generate Ecto models from already existing tables

MIT License

265 stars 54 forks source link

the generated config and the generated schemas had syntax issues or things missing #112

Closed DEvil0000 closed 3 years ago

DEvil0000 commented 4 years ago

when trying this I used it with maria DB. the generated separate config file had the use line missing use Mix.Config. the generated schema files messed up the primary_key option (which should be done differently anyway). however it should be primary_key: false not :primary_key false

9to1url commented 4 years ago

Yes, I also encountered this issue. Smallint column not generated in schema but does shows up on changeset.

jhartwell commented 3 years ago

@DEvil0000 The primary key issue and the missing use Mix.Config issues have been fixed.

@9to1url the issue with smallint has been fixed as well.

Both those fixes are in version 2.4.0. Please let me know if you have any other issues.