#13524 closed (fixed)
Changes to (model) formsets' max_num sematics not listed in the 1.2 release notes
| Reported by: | Ramiro Morales | Owned by: | nobody |
|---|---|---|---|
| Component: | Documentation | Version: | 1.2-beta |
| Severity: | Keywords: | max_num | |
| 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
After a couple of threads in the django-es and django-users mailing lists (that also gave bith to #13521/r13234) it seemed clear we aren't documenting in the release notes the backward incompatible changes implemented in r12872 that affect usage of formsets, model formsets and admin inlines.
The patch attached tries to fix that. Any help reviewing the language used and fleshing it out with more content especially by people more familiar with the changes implemented in such commits will be greatly appreciated.
The patch also:
- Moves a note about the admin inlines JS-powered 'Add another' link from the formsets docs to the admin docs because that seem the place where it belongs. It doesn't even belog in the model formsets docs.
- Add a few tests and fixes typos to the formsets and model formsets I created when trying to understand and confirm the new behavior.
Setting milestone = 1.2.
Attachments (3)
Change History (6)
by , 15 years ago
| Attachment: | 13524.1.diff added |
|---|
by , 15 years ago
| Attachment: | 13524.3.diff added |
|---|
Corrects two more typos, adds a couple grammatical changes for readability
comment:1 by , 15 years ago
| Triage Stage: | Unreviewed → Ready for checkin |
|---|---|
| Version: | 1.1 → 1.2-beta |
comment:2 by , 15 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
First iteration of this patch, enhancements welcome