Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Work in progress support for Rails 4 #2375

Merged
merged 69 commits into from
Sep 24, 2013
Merged

Work in progress support for Rails 4 #2375

merged 69 commits into from
Sep 24, 2013

Conversation

parndt
Copy link
Member

@parndt parndt commented Aug 12, 2013

This is work in progress because it still has failing tests, but only a few!

I'm opening this for transparency and so that people can help! 👍

cc @ugisozols

@simi
Copy link
Member

simi commented Aug 14, 2013

@keram y u not interested in this?

@parndt
Copy link
Member Author

parndt commented Aug 28, 2013

@simi I wish. Unfortunately we can't just merge rails4 support without hundreds of unrelated changes coming along with the commits so we've had to do our best. 😢

@keram
Copy link
Contributor

keram commented Aug 28, 2013

sorry but as parndt wroted above ;/ + this is "just" compatibility, I'm interested in complete "rails4 way".

@parndt
Copy link
Member Author

parndt commented Aug 28, 2013

@simi @keram That's how we do it and have always done it over several years now.. first comes compatibility and then we endlessly refactor it with the help of hundreds of contributors while using it in real life applications. We try to keep change sets small, easily understood and able to be reverted. Happy to merge any commits that are in this fashion and work toward a commonly awesome goal that can be enjoyed and improved by hundreds/thousands of developers.

@keram
Copy link
Contributor

keram commented Aug 28, 2013

@parndt I'm not criticizing or arguing, I just note for clarification.

@parndt
Copy link
Member Author

parndt commented Aug 28, 2013

@keram likewise ❤️

parndt and others added 24 commits September 3, 2013 01:41
@parndt
Copy link
Member Author

parndt commented Sep 23, 2013

@ugisozols ready to go!

ugisozols added a commit that referenced this pull request Sep 24, 2013
Work in progress support for Rails 4
@ugisozols ugisozols merged commit d9e7d4d into master Sep 24, 2013
@ugisozols
Copy link
Member

@parndt ❤️

@parndt parndt deleted the rails4 branch September 24, 2013 21:23
@pusewicz
Copy link

Does it mean we can use the master with Rails 4? 💃

@ugisozols
Copy link
Member

@pusewicz yes! :)

@pusewicz
Copy link

@ugisozols How far is it from 3.0.0 final?

@parndt
Copy link
Member Author

parndt commented Sep 25, 2013

Couldn't say.. We like to use it quite a lot before releasing a final.

@bvsatyaram
Copy link

I am unable to use master with rails 4.
#2417

Am I missing something?

@parndt
Copy link
Member Author

parndt commented Sep 25, 2013

Generating an application with it is different to using it ;)

@bvsatyaram
Copy link

Oops. I am sorry for that. Is there any quick dirty fix to generate an application?

@parndt
Copy link
Member Author

parndt commented Sep 25, 2013

Generate it using Rails and manually add the required gems to the Gemfile.. as we're in development there are a lot of gems you'll need.

Currently:

gem 'refinerycms', github: 'refinery/refinerycms', branch: 'master'
gem 'refinerycms-i18n', github: 'refinery/refinerycms-i18n', branch: 'master'

gem 'friendly_id', github: 'norman/friendly_id', branch: 'master'
gem 'friendly_id-globalize', github: 'norman/friendly_id-globalize', branch: 'master'
gem 'quiet_assets'

# Add support for refinerycms-acts-as-indexed
gem 'refinerycms-acts-as-indexed', github: 'refinery/refinerycms-acts-as-indexed'

gem 'protected_attributes'
gem 'seo_meta', github: 'parndt/seo_meta', branch: 'master'

gem 'globalize3', github: 'svenfuchs/globalize3', branch: 'rails4'
gem 'paper_trail', github: 'airblade/paper_trail', branch: 'master'
gem 'awesome_nested_set', github: 'collectiveidea/awesome_nested_set', branch: 'master'

@bvsatyaram
Copy link

Thank you 👍

bricesanchez added a commit that referenced this pull request Apr 16, 2016
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.

7 participants