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

Hopefully the final fix for Rails 5 compatibility #527

Merged
merged 5 commits into from
Feb 17, 2017

Conversation

iNecas
Copy link
Member

@iNecas iNecas commented Feb 16, 2017

Fixes Issue #415
Fixes Issue #495
Fixes Issue #510
Related to PR #465 and #526

Kudos to @aminariana and @sblackstone

Stephen Blackstone and others added 3 commits February 17, 2017 00:22
Since rails deprecated some behavior that the old code depended on, we
just change it to match what its been doing under the hood all along and
avoid having to do a depth first search at the same time.
@iNecas
Copy link
Member Author

iNecas commented Feb 16, 2017

roses and red 🔴
violets are blue 🔵,
this PR is green 🍏,
I will merge it soon (new version should by out by the end of the week) 🎉

It's not in our power to keep compatibility with ralis 3, 4 and 5
anymore. 3.2 needs to go out
@iNecas
Copy link
Member Author

iNecas commented Feb 18, 2017

apipie-rails-0.4.0 was released, including this change

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.

2 participants