Skip to content
This repository was archived by the owner on May 8, 2022. It is now read-only.

Update README.md#11

Open
khayes-wdas wants to merge 1 commit intojoelmoss:masterfrom
khayes-wdas:patch-1
Open

Update README.md#11
khayes-wdas wants to merge 1 commit intojoelmoss:masterfrom
khayes-wdas:patch-1

Conversation

@khayes-wdas
Copy link
Copy Markdown

Adding require statement

Adding require statement
@sikachu
Copy link
Copy Markdown
Collaborator

sikachu commented Feb 7, 2013

Where do you see this problem? Are you using dynamic_form with Rails? I think Bundler would already require the gem for you since our gem's name and the library's name is the same.

@khayes-wdas
Copy link
Copy Markdown
Author

I thought it was weird as well but after we dropped the gem in our Gemfile and then ran our tests for the offending forms, the error_messages property still was not defined. It was until we put require 'dynamic_form' in our config/application.rb file at the top with some other requires we have in there that it worked.

@sikachu
Copy link
Copy Markdown
Collaborator

sikachu commented Feb 7, 2013

If you do this, does it work?

gem 'dynamic_form', :require => 'dynamic_form'

Also, just to make sure, this gem is listed in the group that will be included in the test right?

@khayes-wdas
Copy link
Copy Markdown
Author

We are upgrading some large Rails 2 apps to Rails 3 and I didn't realize one needed to have Bundle.require in the application.rb. We added that and it worked great, sans :require => 'dynamic_form'

Copy link
Copy Markdown

@mathieujobin mathieujobin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants