Closed Bug 232097 Opened 16 years ago Closed 16 years ago

Landfill URLs should have a landfillbase entity

Categories

(Bugzilla :: Documentation, defect)

defect
Not set

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: 16 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.