stouset / twitter_bootstrap_form_for

A Rails FormBuilder DSL for generating Twitter Bootstrap forms
https://github.com/stouset/twitter_bootstrap_form_for
MIT License
409 stars 110 forks source link

Rails 4.0 Support #81

Closed DavidAllison closed 10 years ago

DavidAllison commented 11 years ago

Since twitter_bootstrap_form_for relies on actionpack ~> 3 it fails to work with Rails 4.0. Any chance we can get the dependency upgraded in the gemspec?

Resolving dependencies............................................................
Bundler could not find compatible versions for gem "actionpack":
  In Gemfile:
    twitter_bootstrap_form_for (>= 2.0.1.0.rc1) ruby depends on
      actionpack (~> 3) ruby

    rails (>= 4.0.0) ruby depends on
      actionpack (4.0.0)
stouset commented 11 years ago

Issue a pull request and I'll be happy to merge it. :)

I honestly haven't used this plugin myself in years (the bootstrap-2.0 branch has never been officially released, as I've never actually had the chance to use it with 2.0), and don't have the time to regularly maintain it.

apfritts commented 11 years ago

I've updated the a forked version for Rails 4 and Bootstrap 3. I'm sort of new to RoR...so comments would be appreciated!

I created pull request: https://github.com/stouset/twitter_bootstrap_form_for/pull/84

stouset commented 11 years ago

Thanks. I away on vacation without a laptop until the 2nd, so you'll have to wait a bit before I can merge. 

--

Stephen Touset stephen@touset.org

On Thu, Aug 15, 2013 at 5:04 PM, AP Fritts notifications@github.com wrote:

https://github.com/stouset/twitter_bootstrap_form_for/pull/84

Reply to this email directly or view it on GitHub: https://github.com/stouset/twitter_bootstrap_form_for/issues/81#issuecomment-22739686

apfritts commented 11 years ago

You go on vacation without a laptop!?!? JK. Enjoy your vacation! :)

c22shen commented 10 years ago

Hi, I tried to install this gem with rails 4, but this dependency issue occurred. I'm just wondering if there is an update about this? Thanks

stouset commented 10 years ago

85 will close this when merged.