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

Forward options to before_validation in acts_as_mappable #10

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

vpuzzella
Copy link

So you can do stuff like:

acts_as_mappable :auto_geocode => {:field => :postal_code}, :on => nil, :if => :postal_code_changed?

@jlecour
Copy link
Contributor

jlecour commented Jan 31, 2012

Hi, I'm very sorry that I didn't take care of this in a loooong time.

I'd like not to have the removal of the _old file in this pull-request. We can argue that this file remains useful or not, but it should not be deleted in a not-related PR.

@jlecour
Copy link
Contributor

jlecour commented Jan 31, 2012

About the patch itself, it should be great to have some tests for the different options.

This reverts commit 515fba4.
@vpuzzella
Copy link
Author

I'll try to whip up some tests when I have time.

@vpuzzella vpuzzella closed this Jan 31, 2012
@vpuzzella vpuzzella reopened this Jan 31, 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

Successfully merging this pull request may close these issues.

2 participants