Skip to content

Conversation

austenito
Copy link

  • helpers like filepicker_js_include_tag aren't available unless they are included when ActionView is loaded.

- helpers like filepicker_js_include_tag aren't available unless they are included when ActionView is loaded.
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 8c49e39 on austenito:patch-1 into 5ca6e25 on Ink:master.

@maurogeorge
Copy link
Contributor

You received some error to make this PR? What version of ruby and rails are tested?

@austenito
Copy link
Author

Hello. Yes I wasn't able to include the gem and have access to the ApplicationHelper methods. I had to include them manually.

I'm using Rails 3.2.3 and Ruby 2.0-p247.

I'm guessing that I am the only one having this issue 💥

@maurogeorge
Copy link
Contributor

You received the error on a view? Could you please paste a stacktrace?

@T145
Copy link

T145 commented Oct 23, 2013

Try upgrading to Rails 4; I've used that and and the same Ruby environment and everything's worked out nicely.

@austenito
Copy link
Author

Upgrading to Rails 4 isn't an option right now :) 


Sent from Mailbox for iPhone

On Wed, Oct 23, 2013 at 8:43 AM, Taylor Shuler notifications@github.com
wrote:

Try upgrading to Rails 4; I've used that and and the same Ruby environment and everything's worked out nicely.

Reply to this email directly or view it on GitHub:
#70 (comment)

@maurogeorge
Copy link
Contributor

@austenito could you please paste a stacktrace? I tested with 1.9.3 and Rails 3.2.x and got no errors.

@austenito
Copy link
Author

There aren't any errors on my end and I added some debugging to verify this code gets called:

https://github.com/Ink/filepicker-rails/blob/master/lib/filepicker_rails/engine.rb#L12

Perhaps I am missing a configuration step? Although if that line is being invoked, then I would expect the methods to be included.

@maurogeorge
Copy link
Contributor

@austenito if no errors, you have the helpers correctly on your views 😄

Follow the README, maybe missing something.

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