Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7k
Primary Support of UniqueConstraint#7438
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
9a23276 to83c23f7Comparekalekseev commentedAug 7, 2020
@carltongibson@tomchristie can we add this to 3.12 milestone? I'm ready to contribute updates to the patch as requested. |
carltongibson commentedAug 7, 2020
Hi@kalekseev. Yes. Super. It's just waiting for a bit of capacity to review. Thanks 👍 |
NyanKiyoshi commentedMar 1, 2021
I believe a test case for the serializer error message would be good to have to ensure the message is as expected. Currently it seems like any message is "good enough" which might cause unwanted/unexpected effects |
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. |
kalekseev commentedMar 27, 2022
It's not stale, still waiting review from maintainers. |
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. |
JorgenPhi commentedMay 31, 2022
Bump |
12c78c9 tofdcd9efComparecarltongibson commentedJun 1, 2022
Just to comment to re-flag this as a change worth having. ORMConstraints are getting increasingly rounded out, and as such are the expected API going forward. |
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. |
kalekseev commentedAug 15, 2022
Not stale, I will rebase as soon as someone is ready to review. Probably can be superseded by#7173 (comment) |
auvipy commentedSep 1, 2022
so drf level validation might not needed? |
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. |
auvipy commentedNov 21, 2022
as we still support django 3.0, so we might still follow the approach in this PR. considering the fact that django orm level validation is available in 4.1 mostly. |
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. |
auvipy left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
if you can fix the merge conflicts it would be really great
fdcd9ef to6ce7d29Comparekalekseev commentedJan 25, 2023
@auvipy rebased |
auvipy commentedJan 25, 2023
I will review it timorrow |
auvipy commentedJan 26, 2023
can we checkhttps://docs.djangoproject.com/en/4.0/ref/models/constraints/ and see if we could cover more test cases/edge cases here? |
kalekseev commentedJan 26, 2023
The pr lack of support for *expressions that were added recently, I will look into it when I have time. |
auvipy commentedMar 1, 2023
If you can handle the new API's in this PR it would be really great. |
kalekseev commentedMar 3, 2023
@auvipy this is in my backlog, unfortunately I'm crushed with work right now, not sure when I'll be able to review it
Is there a work done to introduce support for new django validation into DRF? |
auvipy commentedMar 3, 2023
not really. in that case we can move forward with the current implementation of this patch for now and implement the new API's in another take. |
In DRF 3.15 they added, in the serializer validator, the modelconstraint validation. With the playlist model, we have to force thelti_id default value and manage better in the test the constraintbetween organization or consumer_site existence with the lti_id.Seeencode/django-rest-framework#7438
In DRF 3.15 they added, in the serializer validator, the modelconstraint validation. With the playlist model, we have to force thelti_id default value and manage better in the test the constraintbetween organization or consumer_site existence with the lti_id.Seeencode/django-rest-framework#7438
Uh oh!
There was an error while loading.Please reload this page.
A starting point for discussion, that pr supports both cases when UniqueConstraint consist of one field ->
UniqueValidator, and several fields ->UniqueTogetherValidator. The implementation of single field unique validator is quite naive where we loop over all constraints searching for single field constraints and we do it for each field, would be good to cache that somehow.refs#7173