Opened 10 years ago

Closed 6 years ago

Last modified 5 years ago

#1834 closed enhancement (fixed)

Admin templatetags should have docstrings for admin documentation

Reported by: James Bennett Owned by: Joshua Russo
Component: Documentation Version: master
Severity: normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

The admin templatetags currently have very little in the way of docstrings, which means that they don't provide any useful information in the admin documentation view; these should be fleshed out.

Attachments (5)

adminapplist.diff (633 bytes) - added by James Bennett 10 years ago.
Documentation patch for admin_app_list tag
adminmedia.diff (387 bytes) - added by James Bennett 10 years ago.
Documentation patch for admin_media_prefix tag
include_admin_script_doc.diff (830 bytes) - added by Matias Hermanrud Fjeld <mhf@…> 10 years ago.
documentation for the include_admin_script tag
admin_templatetags_docstrings.diff (2.9 KB) - added by Joshua Russo 6 years ago.
docstrings for admin templatetags and some internal functions
1834-1.1.X.diff (2.8 KB) - added by Ramiro Morales 6 years ago.
Rupe's patch ported to 1.1.X branch

Download all attachments as: .zip

Change History (16)

Changed 10 years ago by James Bennett

Attachment: adminapplist.diff added

Documentation patch for admin_app_list tag

Changed 10 years ago by James Bennett

Attachment: adminmedia.diff added

Documentation patch for admin_media_prefix tag

comment:1 Changed 10 years ago by Adrian Holovaty

Resolution: fixed
Status: newclosed

(In [2920]) Fixed #1834 -- Added docstrings to admin template tags. Thanks, ubernostrum

comment:2 Changed 10 years ago by James Bennett

Resolution: fixed
Status: closedreopened

Adrian, I was actually planning to do docstrings for all of the tags, not just those two. Just haven't had a spare moment to get around to it.

Changed 10 years ago by Matias Hermanrud Fjeld <mhf@…>

documentation for the include_admin_script tag

comment:3 Changed 10 years ago by anonymous

Summary: Admin templatetags should have docstrings for admin documentation[patch] Admin templatetags should have docstrings for admin documentation

comment:4 Changed 10 years ago by Malcolm Tredinnick

(In [3147]) Refs #1834 -- Documented include_admin_script tag in admin interface. Thanks to
Matias Hermanrud Fjeld.

comment:5 Changed 10 years ago by Malcolm Tredinnick

Summary: [patch] Admin templatetags should have docstrings for admin documentationAdmin templatetags should have docstrings for admin documentation

All the current patches are now in svn, so removing the [patch] keyword until some new additions are made.

comment:6 Changed 10 years ago by James Bennett

Owner: changed from Jacob to James Bennett
Status: reopenednew

Claiming this so that, if it's still pending a little ways down the road, I'll remember to do it.

comment:7 Changed 10 years ago by Chris Beaven

Triage Stage: UnreviewedAccepted

ubernostrum, I'm guessing there's still some work to do on this? *nudge*

Changed 6 years ago by Joshua Russo

docstrings for admin templatetags and some internal functions

comment:8 Changed 6 years ago by Joshua Russo

Has patch: set
milestone: 1.2
Owner: changed from nobody to Joshua Russo
Status: newassigned
Version: SVN

This should be good to go

Changed 6 years ago by Ramiro Morales

Attachment: 1834-1.1.X.diff added

Rupe's patch ported to 1.1.X branch

comment:9 Changed 6 years ago by Ramiro Morales

Triage Stage: AcceptedReady for checkin

Last two patched attached to this ticket should be RFC (trunk and 1.1.X).

comment:10 Changed 6 years ago by Brian Rosner

Resolution: fixed
Status: assignedclosed

(In [13139]) Fixed #1834 -- added docstrings to admin templatetags

Thanks Rupe for work on a patch.

comment:11 Changed 5 years ago by Jacob

milestone: 1.2

Milestone 1.2 deleted

Note: See TracTickets for help on using tickets.
Back to Top