Skip to content

Update README.md to demonstrate user creation#132

Merged
olasitarska merged 1 commit intoDjangoGirls:masterfrom
boscoMW:patch-1
Oct 3, 2014
Merged

Update README.md to demonstrate user creation#132
olasitarska merged 1 commit intoDjangoGirls:masterfrom
boscoMW:patch-1

Conversation

@boscoMW
Copy link
Copy Markdown
Contributor

@boscoMW boscoMW commented Oct 3, 2014

Create use step was skipped in the orm demonstration, this update reflects that.

Create use step was skipped in the orm demonstration, this update reflects that.
olasitarska added a commit that referenced this pull request Oct 3, 2014
Update README.md to demonstrate user creation
@olasitarska olasitarska merged commit 7b97d73 into DjangoGirls:master Oct 3, 2014
@olasitarska
Copy link
Copy Markdown
Member

Thanks! :)

@bmispelon
Copy link
Copy Markdown
Contributor

For reference, User.objects.create(...) creates an unusable user (no password and has is_staff=False so it can't connect to the admin).

I don't remember the order of the chapters but if the admin chapter comes after this one then we might run into problems.

@boscoMW
Copy link
Copy Markdown
Contributor Author

boscoMW commented Oct 3, 2014

@bmispelon This user is created with the purpose of using it when creating a Post instance.I've not encountered it further in the tutorial.What you might be referring to is the admin user which is created later.

@bmispelon
Copy link
Copy Markdown
Contributor

@mbosco You're right (there's been some change in the order of chapters and I haven't kept up with it).

My worry was that the attendee would try and create an admin user later with the same name and there'd be an error.
But I checked out the admin chapter and saw that we recommend adding a user called admin so it shouldn't be a problem.

I just wanted to raise this issue in case you had missed it but it seems you're on top of everything so don't mind me :)

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.

3 participants