Opened 18 years ago
Closed 18 years ago
#3302 closed defect (fixed)
[patch] move documention snippet from models-api to django-admin
Reported by: | anonymous | Owned by: | Jacob |
---|---|---|---|
Component: | Documentation | Version: | |
Severity: | normal | Keywords: | sqlinitialdata documentation |
Cc: | Triage Stage: | Ready for checkin | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
Ticket #2953 (http://code.djangoproject.com/changeset/2953) added an an important clarification on how sqlinitialdata works, under the "providing initial sql data" header:
---
Note that if you have multiple SQL data files, there's no guarantee of the order in which they're executed. The only thing you can assume is that, by the time your custom data files are executed, all the database tables already will have been created.
---
However, there is no mention of this behavior in the django-admin documentation on the sqlinitialdata option itself, which is where I would have expected it to be (and would have saved me some time). Since you reference and link to the django-admin sqlinitialdata section documentation in the previous paragraph on models-api page, this paragraph would be more appropriate in the django-admin sqlinitialdata section.
Attachments (1)
Change History (4)
by , 18 years ago
Attachment: | initialsqldoc.diff added |
---|
comment:1 by , 18 years ago
Summary: | move documention snippet from models-api to django-admin → [patch] move documention snippet from models-api to django-admin |
---|
Added patch that added the note to the django-admin reference. However, I think the existing note should stay where it is, as that's where people will be reading about using initial sql.
comment:2 by , 18 years ago
Keywords: | sqlinitialdata documentation added |
---|---|
Triage Stage: | Unreviewed → Ready for checkin |
comment:3 by , 18 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Patch to docs/django-admin.txt