Code review comment for lp:~cjohnston/ubuntu-ci-services-itself/ts-people

Revision history for this message
Andy Doan (doanac) wrote :

157 + email = models.EmailField(max_length=200)

Can you document why that's a shorter length than the name field?

Since this is a "get-only" api, should we add a unit-test to confirm that or will this become a read-write very soon?

« Back to merge proposal