Update models.py #5
Conversation
|
That's great @gtauanearu - thanks! I've added one comment. After you've made a database change, you need to run One other thing I haven't mentioned during sessions is running If you can do those things, commit them, and push them up that would be great! |
| last_name = models.CharField(max_length=100, blank=True) | ||
| country = models.CharField(max_length=100, blank=True) | ||
| mobile_number = models.CharField(max_length=20, blank=True) | ||
| user_email = models.EmailField(max_length=70,blank=True,unique=True) |
There was a problem hiding this comment.
Thanks for the commit!
issue (blocking): if the field is marked as able to be blank, then the unique rule will mean only one user can have a blank email. For now, can you remove the unique=True please?
I would also suggest extending the length to 256 from 70 (as that is the maximum length that an email server will accept for an email address. I should have included that detail in the ticket)
I am adding the user_email address field in the person models!
Ticket: https://trello.com/c/yFlGRZY6/9-15-add-email-address-and-gender-to-person-model