Skip to content

fix for activemodel 3.2 support (still works on 3.0) #10

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix for activemodel 3.2 support (still works on 3.0) #10

wants to merge 2 commits into from

Conversation

lastobelus
Copy link

No description provided.

@lastobelus
Copy link
Author

I didn't commit the Gemfile.lock, so requires a bundle update

@pauldowman
Copy link
Owner

Thank you!! Very much appreciated! I'll merge this in ASAP.

@pauldowman
Copy link
Owner

Sorry, I haven't forgotten, just super busy! (new baby, and busy with work). I'll merge it this weekend.

BTW, I'd love it if someone would take over this project, are you interested in becoming the official maintainer by any chance? I could point everyone to your fork.

@kyletolle
Copy link

I ran into this issue tonight. Until this gets merged into this gem, I've updated my gemspec to point to @lastobelus' repository and branch.

@lastobelus
Copy link
Author

Hi Paul, congrats on your kid. I am actually having my first kid too sometime in december (yea!) so I probably should not take on being official maintainer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Anastasia22zx Anastasia22zx Anastasia22zx approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@lastobelus @pauldowman @kyletolle @Anastasia22zx