#21516 closed Cleanup/optimization (fixed)
Update the import path for the FormSet classes and factories in documentation.
Reported by: | loic84 | Owned by: | Bryan Marty |
---|---|---|---|
Component: | Documentation | Version: | dev |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | yes |
Easy pickings: | yes | UI/UX: | no |
Description
Since #21489, FormSet classes and factories are exposed directly on the django.forms
package.
Examples in the docs should be updated to reflect this change at some point. The docs update is delayed for a couple of releases to help people who misuse the dev docs with an earlier version of Django so they don't get confusing import errors.
The original ticket (#21489) has a tentative patch and discussions relevant to this ticket.
Change History (10)
comment:1 by , 11 years ago
Triage Stage: | Unreviewed → Someday/Maybe |
---|
comment:2 by , 9 years ago
Easy pickings: | set |
---|---|
Triage Stage: | Someday/Maybe → Accepted |
comment:3 by , 9 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:4 by , 9 years ago
This is my first ticket / contribution to the Django project.
I have created a topic branch at:
https://github.com/bxm156/django/tree/ticket_21516
Commit:
https://github.com/bxm156/django/commit/9c117b9bd62ea090f1d8b4d7fb7345ebaaf9e836
Ran make html in docs locally, the build succeeded.
comment:5 by , 9 years ago
Has patch: | set |
---|
comment:6 by , 9 years ago
Patch needs improvement: | set |
---|
This change was made in Django 1.7 and all older versions are now unsupported so it seems like a good time to make the change. See 1c7a83ee8e3da431d9d21dae42da8f1f89973f7c for the functions and classes to update.