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

Add support for Rails 5.2.1 #938

Merged
merged 1 commit into from
Aug 4, 2018

Conversation

y-yagi
Copy link
Contributor

@y-yagi y-yagi commented Jul 31, 2018

Management of alias tracker has changed in rails/rails@50036e6.
So fixed accordingly with it.

@y-yagi y-yagi force-pushed the support-rails-5-2-1 branch from aaaa8da to 8a1f2cf Compare July 31, 2018 12:23
@gregmolnar
Copy link
Member

@y-yagi Thanks for working on this!

Management of alias tracker has changed in rails/rails@50036e6.
So fixed accordingly with it.
@y-yagi y-yagi force-pushed the support-rails-5-2-1 branch from d3f7ac4 to bd55fb1 Compare August 1, 2018 12:01
@y-yagi
Copy link
Contributor Author

y-yagi commented Aug 2, 2018

Seems test all passed. I removed WIP.

@y-yagi y-yagi changed the title [WIP] Add support for Rails 5.2.1 Add support for Rails 5.2.1 Aug 2, 2018
@gregmolnar gregmolnar merged commit e9d5719 into activerecord-hackery:master Aug 4, 2018
@gregmolnar
Copy link
Member

Thank you @y-yagi !

@pedrofurtado
Copy link

@y-yagi Is this PR related to this? #942

if so, please, release a new version of gem with this fix.

Possible related issues:

rails/rails#33557
#942
solidusio/solidus#2825
activerecord-hackery/polyamorous#45
activerecord-hackery/polyamorous#46
activerecord-hackery/polyamorous#41
activerecord-hackery/polyamorous#43

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.

3 participants