Closed Bug 232097 Opened 21 years ago Closed 21 years ago

Landfill URLs should have a landfillbase entity

Categories

(Bugzilla :: Documentation, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 2.16

People

(Reporter: justdave, Assigned: cso)

Details

Attachments

(1 file, 1 obsolete file)

All of the usage examples in the docs tell the user to try things on Landfill. Every lesson in the user guide section gives a link to the relevant page in Bugzilla on Landfill. Obvious example of the problem: In the 2.16 version of the docs, all the links point at landfill/bugzilla-tip. The links in the 2.16 version of the docs should point at http://landfill.bugzilla.org/bugzilla-2.16.4/ (or 2.16.5 once that version is out, etc). Since this is likely to change as versions of the docs change, this should be an entity in the definitions section which can easily be changed.
Attached patch Trunk Patch (obsolete) — Splinter Review
This adds a &landfillbase; entity and uses it.
Comment on attachment 139847 [details] [diff] [review] Trunk Patch Erm, oops. Will try that again.
Attachment #139847 - Attachment is obsolete: true
Attached patch Trunk PatchSplinter Review
Patch, Take 2
Attachment #139848 - Flags: review?(justdave)
Sounds good. We do plan to set up a dedicated Bugzilla for demos on landfill, however. Gerv
(In reply to comment #4) > Sounds good. We do plan to set up a dedicated Bugzilla for demos on landfill, > however. Yes, and when the time comes, having this in place will make it really easy to change those URLs to point at it. And each version of the docs is going to have a different demo URL, and we plan to have 3 different branches in active support at any given time under the new release plan.
Comment on attachment 139848 [details] [diff] [review] Trunk Patch OK, I want someone who knows docbook really good to have a look at this ;) I seem to remember someone mentioning on IRC that we shouldn't be using entities at all but there was some other new trick to use (but maybe I'm remembering the wrong thing).
Attachment #139848 - Flags: review?(justdave)
Attachment #139848 - Flags: review?(gleblanc)
Attachment #139848 - Flags: review?(bz)
Comment on attachment 139848 [details] [diff] [review] Trunk Patch Patch looks good, those are all valid AFAICT, without downloading and really verifying. The method other than entities that is available is to use XInclude. That's sure to break OpenJade. It should work just fine with xsltproc and such tools, when things get moved over to those.
Attachment #139848 - Flags: review?(gleblanc) → review+
Preliminarily moving all docs bugs to 2.18, we should make a valiant attempt to have the docs as up-to-date as possible when 2.18 releases.
Target Milestone: --- → Bugzilla 2.18
Assignee: documentation → bmo
Target Milestone: Bugzilla 2.18 → Bugzilla 2.16
Checked in on both branches (landfillbase points at the 2.16.5 demo on the 2.16 branch). Thanks, Colin!
Status: NEW → RESOLVED
Closed: 21 years ago
Flags: documentation2.16+
Flags: approval2.16+
Flags: approval+
Resolution: --- → FIXED
Comment on attachment 139848 [details] [diff] [review] Trunk Patch Removing r? from FIXED bug.
Attachment #139848 - Flags: review?(bz)
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: