Open tan4eto86 opened 11 years ago
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:
https://github.com/mde/model/pull/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
- M lib/index.js https://github.com/mde/model/pull/99/files#diff-0(5)
Patch Links: