178230 - Update documentation for Enterprise Groups

Status: RESOLVED FIXED

(Bugzilla :: Documentation, enhancement, P2)

Description

[Joel Peshkin]

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

Comment 1

[Joel Peshkin]

Also need to include some usage scenarios

Comment 2

[Joel Peshkin]

Attachment: Very rough outline/start on document

Comment 3

[Joel Peshkin]

Attachment: 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 4

[Dave Miller [:justdave]]

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.

Comment 5

[Joel Peshkin]

Attachment: doc patch rev 2

Comment 6

[Matthew Barnson]

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 :)

Comment 7

[Dave Miller [:justdave]]

Barnboy: wrong checkbox. :)  Go into Edit next to the attachment in question,
and hit the box next to review in there :-)

Comment 8

[Dave Miller [:justdave]]

Comment on attachment 109968 [details] [diff] [review]
doc patch rev 2

per barnboy's comment

Comment 9

[Joel Peshkin]

Attachment: patch with quoting fixed

Comment 10

[Joel Peshkin]

Attachment: One more quoting fix

Comment 11

[Jacob Steenhagen]

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 12

[Joel Peshkin]

Attachment: fixed para tags

Comment 13

[Joel Peshkin]

Comment on attachment 110256 [details] [diff] [review]
fixed para tags

Jake: If you pass this, please add a request for JayPee.

Comment 14

[Jacob Steenhagen]

Comment on attachment 110256 [details] [diff] [review]
fixed para tags

OK, this looks good to me.

Comment 15

[J. Paul Reed [:preed]]

Comment on attachment 110256 [details] [diff] [review]
fixed para tags

Read through the changes for content only; looked good to me. r=preed

Comment 16

[Dave Miller [:justdave]]

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.

Comment 17

[Joel Peshkin]

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