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
Updated for Spree 3.0 stable branch #4
Conversation
@@ -1,7 +1,11 @@ | |||
Spree.config do |config| | |||
# create a list of admin emails | |||
admin_role = Spree::Role.find_by_name('admin') | |||
admin_emails = admin_role.users.map(&:email).join(',') | |||
if ENV['RAILS_ENV'] == 'test' |
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.
Hey @farneman,
Feel its better to not add this to the initializer, since all sites will then have this code. Would prefer to do it in a before
block in the specs.
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.
Maybe its better for the generator to leave all config options commented.
I would agree about leaving all the config options commented—thereby letting each project implement the email list source as they need to. I'll make that change and update the specs to set the config accordingly. |
Groovy! |
@joshnuss I've update the PR now. |
Updated for Spree 3.0 stable branch
Awesomeness, thanks @farneman! |
No description provided.