Re: [Django] #20876: Polls in polls

[Django]

#20876: Polls in polls
-+-
 Reporter:  anonymous|Owner:
 Type:   |  rodolfo2488
  Cleanup/optimization   |   Status:  assigned
Component:  Documentation|  Version:  1.5
 Severity:  Normal   |   Resolution:
 Keywords:  afraid-to-commit | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  1
-+-
Changes (by rodolfo2488):

 * owner:  nobody => rodolfo2488
 * status:  new => assigned


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.f63a304dc0f28fe5bf313f69c6e28471%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #20876: Polls in polls

[Django]

#20876: Polls in polls
-+-
 Reporter:  anonymous|Owner:
 Type:   |  rodolfo2488
  Cleanup/optimization   |   Status:  assigned
Component:  Documentation|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  afraid-to-commit | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  1
-+-
Changes (by timo):

 * has_patch:  0 => 1
 * stage:  Ready for checkin => Accepted


Comment:

 Please don't mark your own patch as Ready for checkin. Someone else needs
 to review it and do that.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.e079f6c2496083360d406d3c8ecd6c94%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #20876: Polls in polls

[Django]

#20876: Polls in polls
-+-
 Reporter:  anonymous|Owner:
 Type:   |  rodolfo2488
  Cleanup/optimization   |   Status:  assigned
Component:  Documentation|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:  afraid-to-commit | Triage Stage:  Ready for
Has patch:  0|  checkin
  Needs tests:  0|  Needs documentation:  0
Easy pickings:  1|  Patch needs improvement:  0
 |UI/UX:  1
-+-
Changes (by rodolfo2488):

 * version:  1.5 => master


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.360f0b9b2b2e756ea3592b28e7980e49%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #20876: Polls in polls

[Django]

#20876: Polls in polls
-+-
 Reporter:  anonymous|Owner:
 Type:   |  rodolfo2488
  Cleanup/optimization   |   Status:  assigned
Component:  Documentation|  Version:  1.5
 Severity:  Normal   |   Resolution:
 Keywords:  afraid-to-commit | Triage Stage:  Ready for
Has patch:  0|  checkin
  Needs tests:  0|  Needs documentation:  0
Easy pickings:  1|  Patch needs improvement:  0
 |UI/UX:  1
-+-
Changes (by rodolfo2488):

 * stage:  Accepted => Ready for checkin


Comment:

 I just made a pull request to this issue at
 [https://github.com/django/django/pull/1574]. The reason of this is that
 the patch file is too big to attach it in the ticket track.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.d15e3e431947844cd7bcd5a6ae6fd724%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #20876: Polls in polls

[Django]

#20876: Polls in polls
--+
 Reporter:  anonymous |Owner:  nobody
 Type:  Cleanup/optimization  |   Status:  new
Component:  Documentation |  Version:  1.5
 Severity:  Normal|   Resolution:
 Keywords:  afraid-to-commit  | Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  1
--+
Changes (by EvilDMP):

 * keywords:   => afraid-to-commit


Comment:

 I agree that this is an issue. I think that the best solution is to follow
 the example of the `Choice`model, so that our two models at
 https://docs.djangoproject.com/en/1.5/intro/tutorial01/#creating-models
 will look like:

 {{{
 class Question(models.Model):
 question_text = models.CharField(max_length=200)
 pub_date = models.DateTimeField('date published')

 class Choice(models.Model):
 poll = models.ForeignKey(Question)
 choice_text = models.CharField(max_length=200)
 votes = models.IntegerField(default=0)
 }}}

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.c1a638a1158e929346ffecd7f417165d%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #20876: Polls in polls

[Django]

#20876: Polls in polls
--+
 Reporter:  anonymous |Owner:  nobody
 Type:  Cleanup/optimization  |   Status:  new
Component:  Documentation |  Version:  1.5
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  1
--+

Comment (by russellm):

 I also agree with the previous two comments, and I make use the "plural
 for app, singular for model" pattern in naming my own apps - but I still
 think the tutorial should be changed.

 We're not talking about general usage here. We're talking about a
 tutorial. This is the user's first experience with Django. Removing any
 source of confusion is a good thing. As it currently stands, we're talking
 about Poll, poll, and polls, and they all refer to different things -- one
 keystroke difference between all three. It's trivial to misread, or to
 become confused as to which object you're using. Grammatically (English
 grammar, not Python), it makes complete sense to say polls.objects.get() -
 so why doesn't that work? After all, I just defined a Poll object, right?
 I make a poll, I retrieve from a list of polls, don't I?

 Once you've established the basic app/model pattern, *then* you can start
 introducing the idea of using pluralization in names.

 If we're not going to rename the polls app, it definitely warrants a
 pullout box that gives an overview of the naming scheme, drawing attention
 to the fact that there are three distinct objects here, all with different
 roles.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.a474c012d83ebe84647cf44170149101%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #20876: Polls in polls

[Django]

#20876: Polls in polls
--+
 Reporter:  anonymous |Owner:  nobody
 Type:  Cleanup/optimization  |   Status:  new
Component:  Documentation |  Version:  1.5
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  1
--+

Comment (by timo):

 I'm inclined to agree with the last two comments and "won't fix", but I'll
 leave open for another opinion given that Russ accepted the ticket.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.df1de8e3580d16e641aa34b9d36e6c6c%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #20876: Polls in polls

[Django]

#20876: Polls in polls
--+
 Reporter:  anonymous |Owner:  nobody
 Type:  Cleanup/optimization  |   Status:  new
Component:  Documentation |  Version:  1.5
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  1
--+

Comment (by saulshanabrook):

 Many of my own apps share the same "problem", (i.e. `artists.Artist`,
 'updates.Update'). I have found this to make sense when an app is really
 focused on a single model.

 Do people consider this an antipattern that we don't want in the tutorial?

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.fbcbf521c27fe96e8c3499560bfcc3a3%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #20876: Polls in polls

[Django]

#20876: Polls in polls
--+
 Reporter:  anonymous |Owner:  nobody
 Type:  Cleanup/optimization  |   Status:  new
Component:  Documentation |  Version:  1.5
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  1
--+

Comment (by koorgoo):

 After the Two Scoops of Django book I prefer to name apps with a plural
 version of the main app's model if I don't see another more obvious
 option.
 The `polls` app's name seems to be clear enough.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.da6a4045c81306c3a587ab17e8adf292%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #20876: Polls in polls

[Django]

#20876: Polls in polls
--+
 Reporter:  anonymous |Owner:  nobody
 Type:  Cleanup/optimization  |   Status:  new
Component:  Documentation |  Version:  1.5
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  1
--+
Changes (by russellm):

 * status:  closed => new
 * resolution:  needsinfo =>
 * stage:  Unreviewed => Accepted


Comment:

 I think the reporter is describing that a project called mysite, with an
 app called polls, with a model called Poll is asking for trouble -- it
 would be better to use a completely unrelated name like "survey" for the
 app, so you are talking about survey.Poll not polls.Poll, making it clear
 that it's not just a matter of pluralization.

 I'm not saying survey is necessarily the best alternative name -- just
 pointing out that a name other than "polls" would be beneficial.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.89f766bd702baee0ca5e5b270b161df3%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #20876: Polls in polls

[Django]

#20876: Polls in polls
-+-
 Reporter:  anonymous|Owner:  nobody
 Type:   |   Status:  closed
  Cleanup/optimization   |  Version:  1.5
Component:  Documentation|   Resolution:  needsinfo
 Severity:  Normal   | Triage Stage:
 Keywords:   |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  1
-+-
Changes (by timo):

 * status:  new => closed
 * needs_better_patch:   => 0
 * resolution:   => needsinfo
 * needs_tests:   => 0
 * needs_docs:   => 0


Comment:

 I assume this is in reference to the tutorial, but the project there is
 called `mysite` so I'm not sure what this is asking for.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.bf526e91a2a53f1743cebb21398088f0%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.