-
Notifications
You must be signed in to change notification settings - Fork 0
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
Basic Emojivision App #13
Conversation
lusbenjamin
commented
Aug 19, 2017
.circleci/config.yml
Outdated
- deploy: | ||
name: maybe deploy | ||
command: | | ||
if [ "${CIRCLE_BRANCH}" == "master" ]; then | ||
if [ "${CIRCLE_BRANCH}" == "basic-form" ]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: put this back to "master"
avoids naming conflict and faster to type
centralized constants file
17fe32b
to
5a7e577
Compare
5a7e577
to
255e368
Compare
app.config['SECRET_KEY'] = 'so-so-secret' # change this | ||
# IMPORTANT: be extremely careful with config['TRAP_HTTP_EXCEPTIONS'] | ||
# Setting to True will break Google App Engine load balancers!!! | ||
# (this probably has to do with GAE expecting a 404 at /_ah/healthcheck) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
^ this comment represents almost 48 hours of debugging
😞
@melodylu FINALLY got this to work on Google App Engine. Going to go ahead and merge this before we review it, but would love to revisit this in detail later. |