#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 by , 10 years ago
Description: | modified (diff) |
---|
comment:2 by , 10 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:3 by , 10 years ago
Keywords: | afraid-to-commit added |
---|
by , 10 years ago
Attachment: | ticket_24244.diff added |
---|
comment:4 by , 10 years ago
Has patch: | set |
---|
comment:5 by , 10 years ago
PR has a test case but no documentation, do we want to separate these into separate tickets?
comment:8 by , 10 years ago
Has patch: | unset |
---|
I committed the test, we can keep this ticket open for the documentation.
comment:9 by , 10 years ago
Owner: | removed |
---|---|
Status: | assigned → new |
comment:10 by , 10 years ago
Where to add documentation ?
Can't find any admin related docs directory in docs/internals
comment:12 by , 10 years ago
Owner: | set to |
---|---|
Status: | new → assigned |
comment:15 by , 10 years ago
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 by , 9 years ago
Owner: | removed |
---|---|
Status: | assigned → new |
When no progress has been made after several months, feel free to take any ticket!
comment:18 by , 9 years ago
Owner: | set to |
---|---|
Status: | new → assigned |
Patch almost ready. Creating PR by tomorrow.
comment:19 by , 9 years ago
Owner: | changed from | to
---|
The pull request is at https://github.com/django/django/pull/5533 . Should I specify it somewhere else?
comment:20 by , 9 years ago
Has patch: | set |
---|
PR: https://github.com/django/django/pull/4363