-
-
Notifications
You must be signed in to change notification settings - Fork 14
feat: add ability to customize model and table name #70
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 4.x
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think everything but that one line looks good.
Fixed 👍 |
@chrisreedio This is a super small PR that is fully backwards compatible. Can we get this one merged? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we're allowing customization of the Connected Account model, should we just depend on that model to set the table name?
Might make the migration kind of "weird" but would prevent any issues of setting the table name to one thing in the config and the model having a different one?
Still thinking about this a bit.
This PR allows you to override the model or just override the table name. This is how Spatie packages handle this (example).
/cc @chrisreedio |
Before this PR the table name was already plural? |
Oh weird. I could have sworn it used to create |
No description provided.