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! |
| @@ -6,3 +6,5 @@ class Person(models.Model): | |||
| 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