Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update README.md to demonstrate user creation #132

Merged
merged 1 commit into from
Oct 3, 2014

Conversation

boscoMW
Copy link
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
Member

Thanks! :)

@bmispelon
Copy link
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
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
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.

None yet

3 participants