Skip to content

Conversation

@DerKobe
Copy link

@DerKobe DerKobe commented Apr 14, 2015

To work in Rails 4.x views (experiences the Problem with Rails 4.2 / not sure if it's also a problem for 4.0 and 4.1).

@hgmnz
Copy link
Owner

hgmnz commented Apr 14, 2015

I haven't used rails in a while, so looking for a few +1s from the community before merging this!

/cc @zapnap @parndt and everyone? :)

@parndt
Copy link
Collaborator

parndt commented Apr 14, 2015

It seems fine in theory, though I wonder if we should make this plugin less aggressive by making people put this in their controllers manually? Or would that be too much?

@DerKobe
Copy link
Author

DerKobe commented Apr 15, 2015

That would be fine, too, I guess. Then just put the code I added in the readme as a hint?

@zapnap
Copy link
Collaborator

zapnap commented Apr 15, 2015

I'd vote for that as well. Seems cleaner / less invasive that way, and easy enough.

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.

4 participants