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
Use future value of ManyToManyField to check if value would change #1271
Conversation
a7b6911
to
b348ffb
Compare
Looks good. It would be ideal to add a unit test which confirms the bug, and perhaps refer to the issue id in the comment? |
@matthewhegarty you're right, I'll do some more work on it, moving to draft, thanks! |
090caa9
to
3489f0f
Compare
@matthewhegarty I've added tests. I noticed that this PR also modifies the API, so would be a candidate to merge along #1278 |
Yes - it would probably affect a lot of users as well, because the I tried to think if there is anyway of achieving this without breaking the API, but couldn't think of a clean way. I'm planning to modify #1278 to avoid breaking the API. |
Any update on this issue? Still using the library from this branch 😬 |
Hi @felixmeziere, glad to know it is working for you, the testing and feedback is highly appreciated. This PR is marked as breaking change as it will change some function signature. We plan to start working on a 3.0 release now that 2.6 is out. |
3489f0f
to
045a64a
Compare
ef1db22
to
4f88558
Compare
add tests and reference the issue added some notes to tests updated changelog
ed9620d
to
cefe1b4
Compare
Hi @matthewhegarty! I need these changes and the branch is now gone. Is it safe to start using |
Hi @manelclos - yes it is a stable branch so you can use it for testing. |
Problem
Issue #1270
How did you solve the problem?
Acceptance Criteria
No tests broke. Specific tests pending based on acceptance of the change.