pow-auth / pow_assent

Multi-provider authentication for your Pow enabled app
https://powauth.com
MIT License
321 stars 50 forks source link

Migration should have `on_delete: :delete_all` for user reference #168

Open danschultzer opened 4 years ago

danschultzer commented 4 years ago

The Ecto migration mix generator has been lifted from the Ecto library where on_delete: :nothing is unfortunately hardcoded:

https://github.com/danschultzer/pow/blob/v1.0.20/lib/pow/extension/ecto/schema/migration.ex#L14

In the case of user identities it really should be on_delete: :delete_all as recommended in the Ecto docs. I'll have to rework the mix generator to permit defaults being passed along with associations.