-
Notifications
You must be signed in to change notification settings - Fork 239
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
Rails 4 or strong_parameters compatible #45
Comments
also deprecation warnings: DEPRECATION WARNING: The following options in your Task.has_many :ancestor_hierarchies declaration are deprecated: :order. Please use a scope block instead. For example, the following:
should be rewritten as the following:
. (called from class:Task at /Users/eturino/developer/rails/tasks/app/models/task.rb:3)
should be rewritten as the following:
. (called from class:Task at /Users/eturino/developer/rails/tasks/app/models/task.rb:3)
should be rewritten as the following:
. (called from class:Task at /Users/eturino/developer/rails/tasks/app/models/task.rb:3)
should be rewritten as the following:
|
I'll start with a branch, but it sure would be nice if the same code worked for rails 3.x and 4.x. I'll see if that can happen. Are you using Rails 4 in production already? |
strong_parameters should be supported already, fwiw. There are tests that cover that support as well. |
I'm not using it in production right now, but I'm starting a new project and I was considering doing it with rails 4 right away, since it'll be a couple of months before it'll be released. Besides, my partner is migrating another app to rails 4 and in a couple of weeks will need hierarchy support and wanted to do it with closure_tree. Thanks a lot! :) |
I looked into it a bit—and wow, it's Rails 2-3 transition pain all over again. OMG Y RAILS Y. I think a rails 4 branch and specific rails-4 version of closure tree looks like what I'm going to have to do. 💩 |
Hi! Any news on the rails 3-4 transition? Cheers! |
Nope, I banged on it for a little bit over the weekend, but it's a fair chunk of work. |
Hi! Is there anything I can help you, mceachen? I'm not familiar with the code, but it seems very clean, I may try to allocate my Rails team for some hours to help. I totally understand your pain. Rails is breaking compatibility on every major release. I'm seriously considering staying on Rails 3.x for a few more months and then create only the NEW projects on 4. That's a headache when you have tenths of projects in Rails. |
@hcastelo, If you check out the https://github.com/mceachen/closure_tree/tree/wip_rails4 branch, you'll see I've only got a few tests still failing—and thanks to @gmanley, I think we'll be able to support rails 3 and rails 4 from the same version (which will be great). I hope to make some more progress tomorrow—but if anyone wants to fix any of the currently failing tests on that branch, pull requests will be more than welcome. |
Rails 4.0.0.rc1 is now supported in version 4.1.0. |
I get this message when I use this gem
Maybe creating a branch or something like that for rails 4 closure_tree ?
The text was updated successfully, but these errors were encountered: