Skip to content

Conversation

@lpatmo
Copy link

@lpatmo lpatmo commented Mar 22, 2020

[x] Change the line in the model to make sure email is required
[x] Make migrations file and run migrations
[x] Make sure tests pass
[x] Double checked in Postman that if I don't include an email, the request is not successful

Implementation details

One-line change:
image

Migration commands I ran:
docker-compose run --rm app ./manage.py makemigrations --check and ocker-compose run --rm app ./manage.py migrate

Postman:
image

The tests pass:
image

I double checked and used the docker commands from @chris48s' codebuddies#109, so am basing this PR off of that PR.

@lpatmo
Copy link
Author

lpatmo commented Mar 22, 2020

Note: this is for documentation purposes only (we don't want to merge this into the testing PR necessarily), so am going to close but keep on the record!

@lpatmo lpatmo closed this Mar 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant