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

Configurable classes for ACL objects and models #148

Open
wants to merge 1 commit into
base: 1.3
Choose a base branch
from

Conversation

TeryaevNikita
Copy link

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.

@intrip
Copy link
Owner

intrip commented Oct 30, 2016

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

@TeryaevNikita
Copy link
Author

I fix some disputable moment

@intrip
Copy link
Owner

intrip commented Nov 1, 2016

Thank you for your PR I'll take a look at it

@intrip
Copy link
Owner

intrip commented Nov 14, 2016

Can you rebase -i to merge multiple commits into one single commit please? 😄

@TeryaevNikita
Copy link
Author

done :)

@TeryaevNikita
Copy link
Author

anything else ?

@intrip
Copy link
Owner

intrip commented Dec 1, 2016

Sorry was busy. I'll take a look at it asap 😄

@erroric
Copy link

erroric commented Jan 23, 2017

+1, add it please

@segsalerty2013
Copy link

Any reason why this is still yet to be merged ? @TeryaevNikita @intrip

@intrip intrip closed this Jul 13, 2017
@intrip intrip reopened this Jul 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants