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

Update README.md #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

khayes-wdas
Copy link

Adding require statement

Adding require statement
@sikachu
Copy link
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
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
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
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

@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