statamic / eloquent-driver

Provides support for storing your Statamic data in a database, rather than flat files.
https://statamic.dev/tips/storing-content-in-a-database
MIT License
104 stars 74 forks source link

[PlanetScale Compatibility] All tables need a primary key #99

Closed jackmcdade closed 1 year ago

jackmcdade commented 1 year ago

The password_resets table has no unique key, which prevents you from promoting a DB branch to production.

CleanShot 2022-11-29 at 14 22 50@2x

Any harm in adding a basic primary, auto-incrementing id column?

jasonvarga commented 1 year ago

password_resets is not part of the eloquent driver, it's in statamic/statamic. There should be no harm in adding it to your own migration though.

fyi that migration is a direct copy from laravel/laravel into statamic/statamic

https://github.com/laravel/laravel/blob/9.x/database/migrations/2014_10_12_100000_create_password_resets_table.php