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

Doesn't work in Rails 3.2.0.rc1! #31

Closed
parndt opened this issue Dec 20, 2011 · 16 comments
Closed

Doesn't work in Rails 3.2.0.rc1! #31

parndt opened this issue Dec 20, 2011 · 16 comments

Comments

@parndt
Copy link
Contributor

parndt commented Dec 20, 2011

I get this lovely error:

/Users/parndt/.rvm/gems/ruby-1.9.3-p0/gems/routing-filter-0.2.4/lib/routing_filter/adapters/rails_3.rb:17:in 'filters': undefined method 'filters' for #Journey::Routes:0x007fa7b5ad2d98 (NoMethodError)

@svenfuchs
Copy link
Owner

hrmmm, i guess that's what you get for doing monster monkey patches :)

wanna give it a try?

@parndt
Copy link
Contributor Author

parndt commented Dec 20, 2011

Honestly I looked first and then filed the issue. :-)

@mjonuschat
Copy link
Collaborator

I've done some initial work to make routing-filter compatible with the new journey router in rails 3.2 - you can see/add/help with the process in my fork using the rails-3.2 branch. With a vey small test application using the pagination and locale filters everything seems to work ok, but I'm battling testing errors on every corner so i'm not too sure about the whole suite.

I've got test failures on 2.3.14 (must be unrelated to my patches), 3.0 is ok, 3.1 and 3.2 don't want to run the testsuite currently.

Gonna submit this as pull request as soon as i'm confident i've whipped everything into shape.

@mjonuschat
Copy link
Collaborator

I think I've whipped everything into shape, tests are all green on ruby 1.8.7, ruby 1.9.2 and ruby 1.9.3 for rails 2.3, 3.0, 3.1 and 3.2

I've sent pull requests #32, #33 and #34 for the different stuff (rails 2.3 bug, test suite and rails 3.2 compatibility)

@svenfuchs
Copy link
Owner

@yabawock wow, thank you for all this work.

i'm currently super busy with travis ci and other things. so if you were interested i'd just add you as a collaborator and you could apply these fixes yourself? wdyt?

@mjonuschat
Copy link
Collaborator

@svenfuchs I'd be honored

@svenfuchs
Copy link
Owner

@yabawock my pleasure :)

welcome on board!

@mjonuschat
Copy link
Collaborator

@parndt
I've merge all fixes into the master branch, so everything should be ok for your Rails 3.2 application. Please go ahead, bundle from master and give it a try.

@kuroda
Copy link
Collaborator

kuroda commented Dec 22, 2011

@yabawock

I will make a gem with beta suffix as soon as possible.

@kuroda
Copy link
Collaborator

kuroda commented Dec 22, 2011

@yabawock

I've pushed routing-filter-0.3.0.beta. Please check if it works well.

Also I wrote CHANGELOG. Modify it if necessary.

@mjonuschat
Copy link
Collaborator

I've updated an app of mine to Rails 3.2.0.rc1 today and all tests are green and a quick check in the browser didn't reveal any oddities either.

@svenfuchs
Copy link
Owner

@yabawock w00t :)

@kuroda
Copy link
Collaborator

kuroda commented Dec 27, 2011

@yabawock

Thanks.

I will remove beta from the version number around 10th January and publish it as a gem if no serious issues are reported.

@svenfuchs
Copy link
Owner

@kuroda thank you so much!

@kuroda
Copy link
Collaborator

kuroda commented Dec 27, 2011

@svenfuchs

Don't mention it! I enjoy making and publishing gems.

@kuroda
Copy link
Collaborator

kuroda commented Jan 10, 2012

I have published a new gem routing-filter-0.3.0. Check it out!

@kuroda kuroda closed this as completed Jan 10, 2012
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

No branches or pull requests

4 participants