Skip to content

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.

2 participants