-
Notifications
You must be signed in to change notification settings - Fork 110
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
Configurable classes for ACL objects and models #148
base: 1.3
Are you sure you want to change the base?
Conversation
Thank you for your PR, I'll take a look at it. Anyways I can't merge in a PR with failing tests: could you try to fix them? Thank you |
I fix some disputable moment |
Thank you for your PR I'll take a look at it |
Can you rebase -i to merge multiple commits into one single commit please? 😄 |
7d85d55
to
73723fd
Compare
done :) |
anything else ? |
Sorry was busy. I'll take a look at it asap 😄 |
+1, add it please |
Any reason why this is still yet to be merged ? @TeryaevNikita @intrip |
Added ability for inheritance and customization of base classes and models. It is necessary for projects with custom users table name and own User class.