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.

Sign up for GitHub

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

Jump to bottom

Require logged in user if navigate to /welcome #3385

Merged
merged 1 commit into from
Mar 22, 2019
Merged

Require logged in user if navigate to /welcome #3385

merged 1 commit into from
Mar 22, 2019

Conversation

microweb10
Copy link
Member

References

This is a backport of PR AyuntamientoMadrid#1928

Objectives

  • Do not raise an exception if a not logged in user navigates to /welcome
  • Tell the user you need to be logged in with a flash message.

@microweb10 microweb10 added the Bug label Mar 19, 2019
@microweb10 microweb10 self-assigned this Mar 19, 2019
@microweb10 microweb10 added this to Review in Roadmap Mar 19, 2019
@microweb10 microweb10 merged commit 9cfd3c7 into master Mar 22, 2019
@microweb10 microweb10 deleted the welcome_page_not_logged_in_user branch March 22, 2019 16:35
@microweb10 microweb10 moved this from Review to Release 0.20 in Roadmap Mar 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@voodoorai2000 voodoorai2000 voodoorai2000 approved these changes

@javierm javierm javierm approved these changes

Assignees

@microweb10 microweb10

Labels
Bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@microweb10 @voodoorai2000 @javierm