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

Adds depend_on_assets directives #1

Closed
wants to merge 1 commit into from
Closed

Adds depend_on_assets directives #1

wants to merge 1 commit into from

Conversation

bdarfler
Copy link

Plays nicely with sprockets_better_errors and Rails 4.1
which require depend_on_assets directives if the asset depends
on other assets. In this case the leaflet.js.erb depends on the
marker images.

Plays nicely with sprockets_better_errors and Rails 4.1
which require depend_on_assets directives if the asset depends
on other assets. In this case the leaflet.js.erb depends on the
marker images.
@bdarfler
Copy link
Author

Looks like this is not the official repo?

@bdarfler bdarfler closed this Apr 22, 2014
@tarmo
Copy link
Owner

tarmo commented Apr 22, 2014

@bdarfler Yeah, correct one is Leaflet/Leaflet

@bdarfler
Copy link
Author

Really? even for the rails gem? I opened a PR against https://github.com/axyjo/leaflet-rails

@tarmo
Copy link
Owner

tarmo commented Apr 22, 2014

Ah right, I forgot the leaflet-rails repo was originally based on Leaflet/Leaflet. It does look like axyjo/leaflet-rails is currently the most active one, I only created my own fork for a one-off need and don't have plans for continuing to maintain it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants