-
Notifications
You must be signed in to change notification settings - Fork 69
Add staging environment #314
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
Conversation
|
||
# Do not dump schema after migrations. | ||
config.active_record.dump_schema_after_migration = false | ||
end |
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 this a copy of production's config?
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.
Yep, we need to change the mail sender though
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.
Will do in another PR
3c1eb1a
to
df9e063
Compare
df9e063
to
f0d061f
Compare
if /^SMTP_(.*)$/ === k | ||
[$1.downcase.to_sym, YAML.load(v)] | ||
end | ||
end.compact] |
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.
Understand this, document it and add staging stuff.
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.
done
This reverts commit 3a85664.
…/staging"" This reverts commit 3a85664.
config/secrets.yml