Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#32570 closed Cleanup/optimization (fixed)

Documentation example for AppConfig is overloaded

Reported by: Aymeric Augustin Owned by: Kshitij Raghav
Component: Documentation Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

In https://docs.djangoproject.com/en/dev/ref/applications/#for-application-authors the example shows default_auto_field = 'django.db.models.BigAutoField' which isn't necessary for the purposes of the example ("here’s how you would provide a proper name for the admin") and isn't explained either. This may create confusion; I suggest we remove this line.

Change History (6)

comment:1 by Mariusz Felisiak, 4 years ago

Easy pickings: set
Triage Stage: UnreviewedAccepted
Type: BugCleanup/optimization

This is a minimal example that doesn't raise error messages. I'm not sure if it's worth removing this single line, but if it's confusing for anyone then we can do this.

comment:2 by Kshitij Raghav, 4 years ago

Owner: changed from nobody to Kshitij Raghav
Status: newassigned

comment:3 by Kshitij Raghav, 4 years ago

Sucessfully removed "default_auto_field = 'django.db.models.BigAutoField' that was causing confusion.

comment:4 by Mariusz Felisiak, 4 years ago

Has patch: set
Triage Stage: AcceptedReady for checkin

comment:5 by Mariusz Felisiak <felisiak.mariusz@…>, 4 years ago

Resolution: fixed
Status: assignedclosed

In d40402c:

Fixed #32570 -- Removed unnecessary default_auto_field in app config example.

comment:6 by Mariusz Felisiak <felisiak.mariusz@…>, 4 years ago

In fb92a9e:

[3.2.x] Fixed #32570 -- Removed unnecessary default_auto_field in app config example.

Backport of d40402cfb023801f0d83f19747e30b13096e3636 from main

Note: See TracTickets for help on using tickets.
Back to Top