search

DjangoGirls / tutorial

This is a tutorial we are using for Django Girls workshops
http://tutorial.djangogirls.org/
Other
1.53k stars 1.86k forks source link

"Django ORM": `User` vs. `user` is confusing #215

Closed encukou closed 9 years ago

encukou commented 9 years ago

Either case-sensitivity needs to be pointed out, or the user variable could just be renamed to e.g. me

bmispelon commented 9 years ago

I think it makes sense to have the variable called me.

olasitarska commented 9 years ago

Always when I see an example like that after Django Girls workshop, I think that it's really obvious that this is confusing. It is sooo hard to put yourself in to beginner's perspective..

Thanks @encukou! Pull Request is really welcome!

miclovich commented 9 years ago

I noticed this in some workshops...students always did something like

User = User.objects.get(username="some_username")

Nice save!

alvynmcq commented 9 years ago

I would be happy to change the user variable to a variable named me.

bmispelon commented 9 years ago

@alvynmcq We'd love to have your contribution too!

The file to change is https://github.com/DjangoGirls/tutorial/blob/master/django_orm/README.md. You should be able to just use Github's online editor (the "edit" icon on the top right) to create the pull request.

Thanks!

alvynmcq commented 9 years ago

@bmispelon I have created the pull request

bmispelon commented 9 years ago

Fixed with #218.

Thank you very much, @alvynmcq :sparkles:

encukou commented 9 years ago

Thanks indeed!