Fixed database update instructions #2116
Conversation
| @@ -44,7 +44,9 @@ When that's all done we'll need to update our database tables. | |||
| Normally we'd create a database migration in order to do that, but for the purposes of this tutorial, let's just delete the database and start again. | |||
|
|
|||
| rm tmp.db | |||
| python manage.py syncdb | |||
| rm -r snippets/migrations | |||
There was a problem hiding this comment.
shouldn't it be mkdirinstead of rm ?
There was a problem hiding this comment.
The directory is already created from previous instructions, we just need to recreate them, that's why the instructions are to remove the migrations and then re-run them.
There was a problem hiding this comment.
Right, missed that point.
We should probably add a note that this isn't a good practice (something such as for the sake of simplicity we do it but don't do that at home). Tutorials are often referred to and I'm not happy with removing the database and migrations to recreate both.
I'm ok for the only reason that doing it the right way will introduce more complexity (default value for the fields).
There was a problem hiding this comment.
I think the line above the snippet already makes that point. When I did the tutorial, I had the idea that this was not a best practice.
Fixed database update instructions
|
Ta! |
As per our discussion here: #2102