Skip to content

use gem dotenv#4

Open
tylersloan wants to merge 3 commits into
ryanmccrary:masterfrom
tylersloan:master
Open

use gem dotenv#4
tylersloan wants to merge 3 commits into
ryanmccrary:masterfrom
tylersloan:master

Conversation

@tylersloan
Copy link
Copy Markdown

This will allow you to store your secret keys, etc, in an ignored file and not have to worry about them while developing locally anymore.

Also updated Ruby version to be compatible with Heroku's Cedar-14 Stack.

Copy link
Copy Markdown
Owner

@ryanmccrary ryanmccrary left a comment

Choose a reason for hiding this comment

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

I just saw this for some reason (I'm getting ready for Christmas this year)

I love these changes.

It wont let me edit before merging - but the only thing I would say to change would be to put

gem 'dotenv-rails'

in the development group of the gemfile so it doesn't break any production setups that are already being used. We set our ENV variables through heroku typically.

Thoughts?

Comment thread app.rb
<script src="https://checkout.stripe.com/checkout.js"></script>
<meta property="og:image" content="http://christmas.goattrips.org/img/400x400.jpg" />
<meta property="og:image:secure_url" content="https://christmas.goattrips.org/img/400x400.jpg" />
<meta property="og:description" content="I participated in GOAT Christmas and you should too! If one person gives each of the values below from $1-200 we will raise just over $20,000 to kickstart our programs for 2015. GOAT would never happen without passionate people giving generously to changing lives in Greenville." />
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

should say "dickstart" our programs

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.

3 participants