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.

Sign up for GitHub

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

Jump to bottom

Loading of backup fails due to nullable natural_key: the USERNAME_FIELD #2247

Closed
Kakadus opened this issue Jul 15, 2024 · 0 comments · Fixed by #2249
Closed

Loading of backup fails due to nullable natural_key: the USERNAME_FIELD #2247

Kakadus opened this issue Jul 15, 2024 · 0 comments · Fixed by #2249
Assignees
@Kakadus
Labels
[C] Backend Focuses on backend implementation [P] Major Major priority [T] Bug This is a bug. We don't like it. Please get rid of it.
Milestone
Winter 2024

Comments

@Kakadus
Copy link
Collaborator

Kakadus commented Jul 15, 2024

Loading a larger backup which contains multiple users with their email None fails with a unique constraint failure.

This should be reproduced by expanding the testdata accordingly and fixed by replacing the default natural key with (self.pk,settings.USERNAME_FIELD)
@Kakadus Kakadus self-assigned this Jul 15, 2024
@Kakadus Kakadus mentioned this issue Jul 16, 2024
Merged
@janno42 janno42 added [C] Backend Focuses on backend implementation [T] Bug This is a bug. We don't like it. Please get rid of it. [P] Major Major priority labels Oct 7, 2024
@janno42 janno42 added this to the Winter 2024 milestone Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Kakadus Kakadus

Labels
[C] Backend Focuses on backend implementation [P] Major Major priority [T] Bug This is a bug. We don't like it. Please get rid of it.
Milestone
Winter 2024
Development

Successfully merging a pull request may close this issue.

Make natural key of userprofile unique Kakadus/EvaP
2 participants
@janno42 @Kakadus