This will be the parent bug for documentation changes related to enterprise group support. This includes.... > 55 (hierarchichal) groups. Rearchitected product groups. Private comments and attachments
Also need to include some usage scenarios
Priority: -- → P2
Target Milestone: --- → Bugzilla 2.18
Created attachment 109967 [details] [diff] [review] documentation patch part 1 This describes the changes from rearchitected product groups It may need further elaboration, but rreplaces incorrect stuff with up-to-date stuff
Comment on attachment 109967 [details] [diff] [review] documentation patch part 1 >+ <command>useentrygroupdefault</command>: > Bugzilla Products can have a group associated with them, so that > certain users can only see bugs in certain products. When this parameter >- is set to <quote>on</quote>, this places all newly-created bugs in the >- group for their product immediately.</para> >+ is set to <quote>on</quote>, this causes the default association >+ between prodycts and groups to place all newly-created bugs in the ^^^^^^^^ s/prodycts/products/ >+ group for their product (same name) immediately.</para> > </step> The resulting above paragraph makes very little sense to me. Care to elaborate a little? (What you're saying sounds like a more wordy way to state the same thing that was already there before) The rest of it looks good to me.
Attachment #109967 - Flags: review?(matthew) → review-
Created attachment 109968 [details] [diff] [review] doc patch rev 2
Attachment #109967 - Attachment is obsolete: true
The content seems correct, but you have a syntax problem. You must use <quote> and </quote>, rather than "something", for syntactically valid Docbook XML. Many parsers transliterate a quote into " for HTML translation, but other ones (notably, PDF and PS) sometimes have strange goofiness when you do it wrong. I know it's wrong elsewhere in the Guide, I aim to clean those up :)
Barnboy: wrong checkbox. :) Go into Edit next to the attachment in question, and hit the box next to review in there :-)
Comment on attachment 109968 [details] [diff] [review] doc patch rev 2 per barnboy's comment
Attachment #109968 - Flags: review?(matthew) → review-
Created attachment 109971 [details] [diff] [review] patch with quoting fixed
Attachment #109968 - Attachment is obsolete: true
Attachment #109980 - Flags: review?(matthew) → review?(preed)
Comment on attachment 109980 [details] [diff] [review] One more quoting fix I should have noticed this when I compiled before, but I didn't. There are some docbook compliation errors with this patch. They all seem to be centered around the fact that the text inside the <listitem/> must also be enclosed in a <para/> tag. >Index: docs//sgml/administration.sgml >=================================================================== >+ <orderedlist> >+ <listitem> >+ required for bug entry, >+ </listitem> Should be: <orderedlist> <listitem> <para> required for bug entry, </para> </listitem> And so on down the line.
Attachment #109980 - Flags: review?(preed) → review-
Created attachment 110256 [details] [diff] [review] fixed para tags
Comment on attachment 110256 [details] [diff] [review] fixed para tags Jake: If you pass this, please add a request for JayPee.
Comment on attachment 110256 [details] [diff] [review] fixed para tags OK, this looks good to me.
Comment on attachment 110256 [details] [diff] [review] fixed para tags Read through the changes for content only; looked good to me. r=preed
nit: what's the behaviour of useentrygroupdefault if makeproductgroups is off? If it's ignored in that situation, then it probably ought to say so in the docs. I won't let that stop checkin though. If you want to fix that before checking in, go for it, just ensure that it compiles.
Flags: approval? → approval+
checked in further additions will be needed and will be done under another bug.
Status: ASSIGNED → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.