Adding UserPassesTestMixin to work like user_passes_test decorator #76

Closed
wants to merge 3 commits into
from

Projects

None yet

4 participants

@galuszkak
Contributor

Django braces for me lacks of user_passes_test for CBV so I create one. Added also test and docs.

If implementation is wrong or I should something get to consideration please feel free to tell me.

Cheers
Kamil Gałuszka

@galuszkak
Contributor

@chrisjones-brack3t I will be grateful for some comments/feedback. :)

@chrisjones-brack3t
Member

I will get @kennethlove to go over this with me next week.

@galuszkak
Contributor

@chrisjones-brack3t Thanks. I appreciate your time and this amazing work with django-braces!

@mrmmm
mrmmm commented Nov 5, 2013

Looking forward for this feature.

@galuszkak
Contributor

@chrisjones-brack3t @kennethlove I've also updated travis and tox removing django 1.3 support and adding django 1.6 support (there was only beta support in tests).

Test on django 1.6 are working.

@galuszkak
Contributor

@chrisjones-brack3t @kennethlove ping. Soory to bother but do you have any ETA to review this pull request? :)

@kennethlove
Member

I just prepared this for release. Got a few more things I want to get in first but this'll be in the upcoming 1.3 release.

@kennethlove kennethlove closed this Jan 3, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment