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

Update hasmany association with through option to work when one of insta... #99

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tan4eto86
Copy link

Update hasmany association with through option

…stances has more than one mapped instanses of the second model.
@mde
Copy link
Contributor

mde commented Sep 5, 2013

This looks great. Could you add a test for this, so it doesn't get
regressed?

On Thu, Sep 5, 2013 at 6:05 AM, tan4eto86 notifications@github.com wrote:

Update hasmany association with through option


You can merge this Pull Request by running

git pull https://github.com/tan4eto86/model master

Or view, comment on, or merge it at:

#99
Commit Summary

  • Update hasmany association with through option to work when one of
    instances has more than one mapped instanses of the second model.

File Changes

Patch Links:

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.

None yet

2 participants