Opened 9 years ago

Closed 9 years ago

#5340 closed (fixed)

request for new link in docs

Reported by: rogerpack2005@… Owned by: justinlilly@…
Component: Documentation Version: master
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

I think
that http://www.djangoproject.com/documentation/templates/ should link to
http://www.djangoproject.com/documentation/templates_python/ at the bottom--maybe a link like "click here to program your own" or what not. Thanks! Django rox!

Attachments (3)

templates.txt (45.2 KB) - added by justinlilly@… 9 years ago.
Proposed patch to fix the escaping issue causing the bug
template.diff (177 bytes) - added by Justin Lilly <justinlilly@…> 9 years ago.
This should be the .diff file required.
template.2.diff (593 bytes) - added by Justin Lilly <justinlilly@…> 9 years ago.
This should be a viewable version of the previous patch.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 9 years ago by Adrian Holovaty

Resolution: fixed
Status: newclosed

(In [6070]) Fixed #5340 -- Linked docs/templates.txt to docs/templates_python.txt

comment:2 Changed 9 years ago by john.scott.atlakson@…

Patch needs improvement: set
Resolution: fixed
Status: closedreopened

This patch results in an error...see http://www.djangoproject.com/documentation/templates/#id3
I don't know much about reStructured Text formatting but my guess that the colon within the link name might be confusing the parser. The line in question:

.. _The Django template language: For Python programmers: ../templates_python/

Changed 9 years ago by justinlilly@…

Attachment: templates.txt added

Proposed patch to fix the escaping issue causing the bug

comment:3 Changed 9 years ago by justinlilly@…

Patch needs improvement: unset
Resolution: fixed
Status: reopenedclosed
Triage Stage: UnreviewedReady for checkin

Should be fixed. Just needed an escaping backslash on the link. I've tested this with http://www.hosting4u.cz/jbar/rest/rest.html which seemed to do okay.

comment:4 Changed 9 years ago by Russell Keith-Magee

Resolution: fixed
Status: closedreopened
Triage Stage: Ready for checkinAccepted

This isn't ready for checkin - it needs to be submitted as a patch, not as a replacement file. Also, please don't mark the ticket as fixed - that is done automatically by SVN during the checkin process.

comment:5 Changed 9 years ago by justinlilly@…

Has patch: set
Owner: changed from Jacob to justinlilly@…
Status: reopenednew

Terribly sorry for that. Well then, I suppose this is me submitting the item as "Has Patch", accepting it personally. Sorry, this is my first ticket so here goes again, eh?

comment:6 Changed 9 years ago by Russell Keith-Magee

Perhaps I wasn't clear - the file you upload needs to be a patch - i.e., a diff file, not a complete replacement for the original.

Changed 9 years ago by Justin Lilly <justinlilly@…>

Attachment: template.diff added

This should be the .diff file required.

Changed 9 years ago by Justin Lilly <justinlilly@…>

Attachment: template.2.diff added

This should be a viewable version of the previous patch.

comment:7 Changed 9 years ago by Malcolm Tredinnick

Resolution: fixed
Status: newclosed

(In [6184]) Fixed #5340 -- Fixed a broken link. Thanks, Justin Lilly.

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