Added slug and uuid field in AppUserModel #378
Conversation
|
earlier I migrated the model with a slug field only. Ig that's causing the issue here, I tried deleting migrations file but it couldn't help. |
| id = models.UUIDField(primary_key=True, default=uuid.uuid4, editable=False) | ||
| slug = models.SlugField(unique=True, blank=True, null=True) |
There was a problem hiding this comment.
Hey @Yadavanurag13 , thankyou for this PR. I have a small comment.
Id field is already there in the the AbstractZangoUserModel and we don't need to override it. We needed a field of type uuid to track app users across our application and its not easy to guess(unlike id field). The reason we named the field slug is because we are going to use it to identify a user.
So, can you please make the below changes, create new migrations and push the latest changes?
| id = models.UUIDField(primary_key=True, default=uuid.uuid4, editable=False) | |
| slug = models.SlugField(unique=True, blank=True, null=True) | |
| slug = models.UUIDField(default=uuid.uuid4, editable=False) |
|
@DevilsAutumn I have made the required changes. Please review and let me know if any further modifications are required. Thanks |
Added uuid and slug field in AppUserModel.
Added migrations file too...
@deepakdinesh1123 On running the command 8000
python manage.py migrateit is giving "django.db.utils.ProgrammingError: column "slug" of relation "appauth_appusermodel" already exists" This error can you please tell me how to resolve this issue. I tried to fix it but couldn't.