Skip to content
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

Support for pluralized table names #9

Open
benmccallum opened this issue Apr 14, 2018 · 2 comments
Open

Support for pluralized table names #9

benmccallum opened this issue Apr 14, 2018 · 2 comments
Assignees

Comments

@benmccallum
Copy link
Contributor

Could we support pluralized table names for all the generic methods like Insert<T>? I see it just defaults to a table with a name the same as type T. I prefer using Users as my table names instead of User.

We could use an attribute on the T classes, like TableName("Users") that is checked before the default is used. What do you think? Or go down a fluent-style declaration in Startup.cs with some extension methods. Or support both even.

@dougrday
Copy link
Member

I prefer fluent style, because it doesn't require you to have control over the source. In some projects, models are out of your control.

@benmccallum
Copy link
Contributor Author

ALmost finished a PR for this. Will submit it when I'm back again haha.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants