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

Redirect AnonymousUser to login page when accessing /assignments/*/new/ #246

Merged
merged 2 commits into from
Nov 28, 2018
Merged

Redirect AnonymousUser to login page when accessing /assignments/*/new/ #246

merged 2 commits into from
Nov 28, 2018

Conversation

johenning
Copy link
Contributor

Quick fix to avoid Exception: 'AnonymousUser' object has no attribute 'profile'
Fixed #245

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 81.026% when pulling 6b0e951 on johenning:loginrequired into 1b53ac8 on troeger:master.

@troeger
Copy link
Owner

troeger commented Nov 28, 2018

Agreed from both perspectives: This is a valid quick fix, and it is not the correct solution. We keep the issue open to not forget the underlying problem.

@troeger troeger merged commit 8583c96 into troeger:master Nov 28, 2018
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.

AnonymousUser accessing /assignments/1/new/ raises exception
3 participants