Skip to content
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

Remove railtie group #18

Closed
wants to merge 1 commit into from
Closed

Remove railtie group #18

wants to merge 1 commit into from

Conversation

riyad
Copy link

@riyad riyad commented Nov 16, 2012

This basically reverts 91a5a02. Otherwise the gemoji images directory won't be added to the assets path.

@josh
Copy link
Contributor

josh commented Nov 16, 2012

Sorry, this breaks non initialize precompile setups like heroku.

@josh josh closed this Nov 16, 2012
@riyad
Copy link
Author

riyad commented Nov 16, 2012

So what else could I do to make it work in my case? What does setting the group on initializer change that I might be missing?

@josh
Copy link
Contributor

josh commented Nov 16, 2012

Add Emoji.path to your config.assets.path if you can't figure it out.

@riyad
Copy link
Author

riyad commented Nov 16, 2012

Thanks. :) Added a Rails initializer ti fix it. But doesn't this defeat the purpose?

My::Application.config.assets.paths << Emoji.images_path

Maybe you should add this to the docs in case someone else has the same problem.

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.

None yet

2 participants