#24244 closed New feature (fixed)
Document contrib.admin.models.LogEntry
| Reported by: | Tim Graham | Owned by: | Alex Morozov |
|---|---|---|---|
| Component: | Documentation | Version: | dev |
| Severity: | Normal | Keywords: | afraid-to-commit |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | yes | UI/UX: | no |
Description (last modified by )
It would be useful to document the fields and methods of this model.
A test also needs to be added for the get_edited_object() method as it's currently not covered.
Attachments (1)
Change History (24)
comment:1 Changed 9 years ago by
| Description: | modified (diff) |
|---|
comment:2 Changed 9 years ago by
| Owner: | changed from nobody to Varun Sharma |
|---|---|
| Status: | new → assigned |
comment:3 Changed 9 years ago by
| Keywords: | afraid-to-commit added |
|---|
Changed 9 years ago by
| Attachment: | ticket_24244.diff added |
|---|
comment:4 Changed 9 years ago by
| Has patch: | set |
|---|
comment:5 Changed 9 years ago by
PR has a test case but no documentation, do we want to separate these into separate tickets?
comment:8 Changed 9 years ago by
| Has patch: | unset |
|---|
I committed the test, we can keep this ticket open for the documentation.
comment:9 Changed 9 years ago by
| Owner: | Varun Sharma deleted |
|---|---|
| Status: | assigned → new |
comment:10 Changed 9 years ago by
Where to add documentation ?
Can't find any admin related docs directory in docs/internals
comment:12 Changed 9 years ago by
| Owner: | set to sholleman |
|---|---|
| Status: | new → assigned |
comment:13 Changed 9 years ago by
| Owner: | changed from sholleman to maestrofjp |
|---|
I have documentation in the works.
comment:14 Changed 9 years ago by
| Owner: | changed from maestrofjp to sholleman |
|---|
Assigning back to me per maestrofjp.
comment:15 Changed 9 years ago by
Apologies for stealing the ticket. I should have refreshed the page after letting it sit for ages. Let we were able to sit down to see that we were both on the same page on how to do it.
comment:17 Changed 8 years ago by
| Owner: | sholleman deleted |
|---|---|
| Status: | assigned → new |
When no progress has been made after several months, feel free to take any ticket!
comment:18 Changed 8 years ago by
| Owner: | set to Varun Sharma |
|---|---|
| Status: | new → assigned |
Patch almost ready. Creating PR by tomorrow.
comment:19 Changed 8 years ago by
| Owner: | changed from Varun Sharma to Alex Morozov |
|---|
The pull request is at https://github.com/django/django/pull/5533 . Should I specify it somewhere else?
comment:20 Changed 8 years ago by
| Has patch: | set |
|---|
PR: https://github.com/django/django/pull/4363