-
Notifications
You must be signed in to change notification settings - Fork 766
Added auto M2M #435
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
Added auto M2M #435
Conversation
1 similar comment
@sciyoshi Any news ? |
Hi @ZuluPro, this looks fine to me however I'm not a maintainer for this project so I'm unable to merge. You'll need to ask @syrusakbary or someone else with commit access to take a look. |
Can we get this merged? |
@syrusakbary Any news |
I like the PR and would love to merge, but we need tests testing the new changes, so we assure the issue is not replicated in future releases. |
@ZuluPro mind adding some tests? |
@ZuluPro interested to work on the tests? |
I'd like to tidy up some of the PRs, so if we still want to work on this let me know or I will close this in 1 week. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Maybe it isn't the correct solution, but I wasn't able to have my
ManyToManyField
used.Without this patch I receive:
{ "errors": [ { "message": "Cannot query field \"mymanyfield\" on type \"MyModelWithManyNode\".", "locations": [ { "line": 4, "column": 5 } ] } ] }
I made this patch from 2.0.0 as 2.0.1 doesn't work with my config.