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

Password protect partnered events #407

Closed
wants to merge 4 commits into from

Conversation

ash6851
Copy link
Contributor

@ash6851 ash6851 commented Sep 21, 2016

backend code for #402. front end isn't pretty :)

@@ -25,7 +26,7 @@
SECRET_KEY = 'e^u3u$pukt$s=6#&9oi9&jj5ow6563fuka%y9t7i*2laalk^l$'

# SECURITY WARNING: don't run with debug turned on in production!
DEBUG = str_to_bool(os.environ.get('DEBUG')) or False
DEBUG = str_to_bool(os.environ.get('DEBUG')) and 'test' not in sys.argv or False
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ash6851 Can you explain what this addition does?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@karbassi when I was writing tests I was trying to check response url, but the redirect was being intercepted by the redirect debug page and my response url was wrong. Let me know if you need more explanation

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ash6851 ah, yes. This needs to be set to default to false. I'll commit a fix ASAP.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ash6851 Merge in #409

@karbassi karbassi changed the title Feature/password protect Password protect partnered events Oct 6, 2016
@karbassi
Copy link
Member

karbassi commented Oct 7, 2016

Great work @ash6851

This was merged in #424 since I couldn't commit my changes to your branch.

Github added a feature that solves this issue. On your next PR, make sure the Allow edits from maintainers is checked.

screen shot 2016-10-06 at 7 57 36 pm

@karbassi karbassi closed this Oct 7, 2016
@karbassi karbassi removed the to review label Oct 7, 2016
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.

2 participants