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

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.

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 &quot; 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

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.

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.

checked in further additions will be needed and will be done under another bug.