Skip to content

Conversation

@edemocracy
Copy link

add support for builtin login to the authorization form
and add AuthorizationLoginView which enables it

and add AuthorizationLoginView which enables it
@edemocracy
Copy link
Author

I'd prefer to make this the default behavior, and to remove LoginRequiredMixin from BaseAuthorizationView, but this would be backwards incompatible. What do you think?

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 89f1834 on edemocracy:feature/authlogin into 0b3712e on evonove:master.

@masci masci added this to the 0.9.0 milestone Jan 7, 2015
@palazzem palazzem modified the milestones: 0.9.0, 0.10.0 Oct 15, 2015
@outsbart
Copy link
Contributor

Thanks for your pull request. Altough this is an interesting way of solving the problem, I believe we should adopt a more configurable and already proven-to-be-good approach instead of using a non-standard one.

@outsbart outsbart closed this Oct 22, 2015
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.

5 participants