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 nil values #1

Merged
merged 1 commit into from May 16, 2016

Conversation

Projects
None yet
2 participants
@westonganger
Copy link

commented Apr 11, 2015

Add support for nil values on associations

I always want this feature thought I would make a pull request so at minimum the information feature is easily found but yould love for you to merge this to master.

All credits to @enriquetuya

@straydogstudio

This comment has been minimized.

Copy link
Owner

commented Jun 3, 2015

@westonganger Sorry so slow. I just want to really dig for implications first, and haven't had time to do anything.

@westonganger

This comment has been minimized.

Copy link
Author

commented Jun 3, 2015

enriquetuya's fork had some tests to go with it as well, you can see the diff here: randym/acts_as_xlsx@master...enriquetuya:master

@straydogstudio straydogstudio merged commit 94fc6aa into straydogstudio:master May 16, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.