#30688: Clarify docs for Options.base_manager_name.
--------------------------------------+------------------------------------
     Reporter:  agilgur5              |                    Owner:  agilgur5
         Type:  Cleanup/optimization  |                   Status:  assigned
    Component:  Documentation         |                  Version:  master
     Severity:  Normal                |               Resolution:
     Keywords:                        |             Triage Stage:  Accepted
    Has patch:  1                     |      Needs documentation:  0
  Needs tests:  0                     |  Patch needs improvement:  1
Easy pickings:  0                     |                    UI/UX:  0
--------------------------------------+------------------------------------
Changes (by Carlton Gibson):

 * needs_better_patch:  0 => 1
 * version:  2.2 => master
 * type:  Uncategorized => Cleanup/optimization
 * stage:  Unreviewed => Accepted


Comment:

 Thanks for this! Yes, but as per comment on PR, I think a clarification of
 the
 
[https://docs.djangoproject.com/en/2.2/ref/models/options/#django.db.models.Options.base_manager_name
 Options.base_manager_name] description is what's needed.

 Currently, "The name of the manager..." is ambiguous as you say. Narrowing
 that to "The attribute name...", adding perhaps "... for example
 `'objects'`..." if we think that's clearer should be enough.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/30688#comment:2>
Django <https://code.djangoproject.com/>
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/066.183562b830cc63ef3717fc3cdd2fbada%40djangoproject.com.

Reply via email to