Opened 18 years ago
Closed 11 years ago
#5405 closed New feature (fixed)
Support restructured text in model docstrings
| Reported by: | Matt McClanahan | Owned by: | Žan Anderle |
|---|---|---|---|
| Component: | contrib.admindocs | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | karld@…, andrew.tennikoff@…, tomas.ehrlich@…, gokmen@…, Žan Anderle, info+coding@… | 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
It'd be nice to be able to use ReST in the docstrings of models. The admindocs views currently parse ReST for template tags, filters, and views, but not models.
Attachments (3)
Change History (30)
by , 18 years ago
| Attachment: | 5405-1.diff added |
|---|
comment:1 by , 18 years ago
| Has patch: | set |
|---|
comment:2 by , 18 years ago
| Triage Stage: | Unreviewed → Ready for checkin |
|---|
Since the other things in admin_docs support rst, then we should be consistent.
comment:3 by , 18 years ago
| Keywords: | nfa-someday added |
|---|
This ticket isn't critical for merging newforms-admin into trunk. Tagging with nfa-someday. Looks like this would be a good improvement and make things consistent.
comment:4 by , 17 years ago
| milestone: | → 1.0 |
|---|
comment:5 by , 17 years ago
| milestone: | 1.0 → post-1.0 |
|---|---|
| Triage Stage: | Ready for checkin → Accepted |
| Version: | newforms-admin → SVN |
comment:6 by , 17 years ago
| Cc: | added |
|---|
comment:7 by , 17 years ago
| Cc: | added |
|---|
comment:9 by , 15 years ago
| Severity: | → Normal |
|---|---|
| Type: | → New feature |
comment:10 by , 15 years ago
| Easy pickings: | unset |
|---|---|
| Needs tests: | set |
comment:13 by , 13 years ago
| Cc: | added |
|---|---|
| Component: | contrib.admin → contrib.admindocs |
comment:14 by , 13 years ago
Also duplicated by #17599 which has very similar patch. I'll post updated version soon.
by , 13 years ago
| Attachment: | 5405-view-detail.diff added |
|---|
comment:15 by , 13 years ago
Patch updated, but I'm not sure how to write tests. There're no similar tests in django.contrib.admindocs.tests. I renamed body in original patch to description to keep the backward compatibility (in case someone has it's own version of model_detail.html).
Also, the ticket #17599 is talking about model_detail only, but there's another misleading information in docs, about using :model: reference in view function. It didn't work for me and I've found there's a striptags filter, which strips all generated links. It's included in the second patch.
comment:17 by , 13 years ago
| Patch needs improvement: | unset |
|---|
comment:18 by , 13 years ago
https://github.com/django/django/pull/760
i need to review to improve tests.
comment:19 by , 13 years ago
| Cc: | added |
|---|---|
| Keywords: | sprint2013 added |
| Owner: | changed from to |
| Status: | new → assigned |
comment:20 by , 12 years ago
| Needs documentation: | set |
|---|---|
| Needs tests: | unset |
| Patch needs improvement: | set |
I left comments on the pull request but it still needs some improvement including a rebase to merge cleanly. It would also be good to mention the change in the release notes.
comment:21 by , 12 years ago
| Easy pickings: | unset |
|---|
comment:24 by , 11 years ago
| Cc: | added |
|---|---|
| Owner: | changed from to |
comment:26 by , 11 years ago
| Cc: | added |
|---|---|
| Keywords: | nfa-someday sprint2013 removed |
| Needs documentation: | unset |
| Patch needs improvement: | unset |
| Triage Stage: | Accepted → Ready for checkin |
The patch looks good to me.
comment:27 by , 11 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
There's a possibility that this patch breaks backwards compatibility if a user overrides the
admin_doc/model_detail.htmltemplate.