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 m2m fields to proccess #22

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

Conversation

AlexProfi
Copy link

Hello please view this functionality
I also make logic for scrapy parser to use

@MalikRumi
Copy link

Alex
I just ran into this problem, and haven't found much documentation. Maybe if we could explain more to the core developers what the problem is, there would be more activity and perhaps acceptance of your code. For your part, it would probably help to include your tests on this code, not only that it works but that it doesn't mess anything else up. I see the big 'this branch has no conflicts' sign, but I don't know what that really means. Test results would surely help.
For myself, it never occurred to me that there would be a problem with M2M fields. I just assumed they would work like FKs. If you could explain why this hasn't been tackled up to now, it might help Alex write the code in a way you would find acceptable. Apparently there hasn't been much demand, which surprises me, but so what? I need it now. Thanks.

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