-
Notifications
You must be signed in to change notification settings - Fork 212
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
Codespaces #3279
Codespaces #3279
Conversation
Frustrating: Bundles. You try to curl elasticsearch. Name no longer resolves! |
Okay, restarting everything made it much more stable! |
APP_PROTOCOL: http | ||
DEVISE_SECRET_KEY: secret | ||
GROWSTUFF_EMAIL: "[email protected]" | ||
GROWSTUFF_FLICKR_KEY: secretkey" |
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.
Is there a missing or extra quote mark here?
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.
wait, shouldn't merging have waited for an answer to this?
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.
I figured it was minor enough that it doesn't matter so long as the overall solution works, or indeed if everything in a codespace works except Flickr, that's not worth stopping merging.
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.
Happy to merge, thank you so much for finishing this off.
Was #3216
Squash merge!