Add UUIDfield to User model - Githubissues

openoakland / TrashTalk

Repo for artifacts of the Trash Talk app for organizing cleanups and graffiti abatement.

Other

7 stars 10 forks source link

Add UUIDfield to User model #161

Open mosdevly opened 6 years ago

mosdevly commented 6 years ago

The abstract User needs a field: uid. It would be nice to use the short 22 char UUID field. You can checkout the resources below for details about this and weigh any pros/cons.

Django UUIDField: https://docs.djangoproject.com/en/2.0/ref/models/fields/#uuidfield
ShortUUIDField: https://github.com/benrobster/django-shortuuidfield

This update requires:

[ ] New field: User.uid that auto creates a uuid when a new user is created.
[ ] Migration
[ ] User API Views: use the new uid as the lookup_field
[ ] urls.py: routes need to use the new uid
[ ] User Serializer: Add the new field to it.
[ ] Any tests that require updating.

Extra mile:

[ ] While you're updating urls, change the relevant routes to use the new path function instead of url: https://docs.djangoproject.com/en/2.0/ref/urls/#path