-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Align option class names #1919
Align option class names #1919
Conversation
Please don't use Abp prefix. Because these classes are already under Abp namespace. |
We needed to add Abp prefix for some option classes those have potential to make developers confusing (since they have same/similar names of asp.net core's options). Then we have a situation where some of them have Abp, while some of them haven't. I am still not sure about this (it is also a breaking change, while it may be reasonable since v1.0 has not been released yet). |
Thanks @hikalkan |
resolves #1176