Sean Rich [:srich] Reporter
	Description • 10 years ago

The WebOps team wants to create a WebOps Request Form in Bugzilla that would take the answers to a number of questions and automatically file a single bug using the contents of the form.

We are trying to improve our ability to respond to requests by simplifying the request while improving our understanding of the request.

Tests:

Given that the new form exists and that a new Component called "WebOps: Request" exists, when the requestor creates a new bug and selects the "Infrastructure & Operations :: WebOps: Request" product/component, the new form will be displayed.

Given that the new form is displayed, only the following field should be displayed:
1. Summary
2. What are you asking us to do?
3. What is the problem you are trying to solve?
4. How would you solve this problem?  How has this problem been solved in the past?
5. Who might be impacted by this change?
6. What are the known dependencies for this request?
7. CC
8. Depends on
9. Blocks

Given that all the fields are displayed, the answers to questions 2, 3, 4, 5, and 6 should be stored in the standard “Description” field, along with their question.  For example:

Description:
What are you asking us to do?
Please renew SSL cert.

Given that the form is filled out, when the requestor presses the “Submit Bug” button the form contents will be saved to a new bug.

Given that the bug is created, the Summary field should be populated (required), the Reporter field should be automatically populated with the email address of the requestor and answers to questions 2 and 3 should be populated (required).  All other field values should be stored as optional in their standard fields.

Please see attached picture for more information about what fields should not be included on the form (x'd out).

	Sean Rich [:srich] Reporter
	Comment 1 • 10 years ago

Dylan, I don't want this to block on me if you have questions, so just hit up jakem, gozer or one of the WebOps folks for help if I don't respond. I'm on PTO from Jul 19-27.  Thanks!

	Dylan Hardison [:dylan] (he/him) Assignee
	Comment 2 • 10 years ago

This is about ready to go up to dev, after I tweak a few minor things.

And a question for dkl:

It is my understanding that forms require a separate entry point -- which means I'm not sure I can do the "when the requestor creates a new bug and selects the "Infrastructure & Operations :: WebOps: Request" product/component, the new form will be displayed" behavior.

	David Lawrence [:dkl]
	Comment 3 • 10 years ago

(In reply to Dylan William Hardison [:dylan] from comment #2)
> This is about ready to go up to dev, after I tweak a few minor things.
> 
> And a question for dkl:
> 
> It is my understanding that forms require a separate entry point -- which
> means I'm not sure I can do the "when the requestor creates a new bug and
> selects the "Infrastructure & Operations :: WebOps: Request"
> product/component, the new form will be displayed" behavior.

Unfortunately we cannot do it on the component level, just product for now. And it would require a code change in extensions/BMO/lib/Data.pm

http://git.mozilla.org/?p=webtools/bmo/bugzilla.git;a=blob;f=extensions/BMO/Extension.pm;h=b651caf6b80ec8858a93fe2ba2779912beed014e;hb=HEAD#l1211 [github]

FWIW, something that could solve this and other forms should be discussed in bug 1037663 as we need a solution.

dkl

	Sean Rich [:srich] Reporter
	Comment 4 • 10 years ago

Guys, I'm not fussed.  If I understand correctly, this new form will be presented for all WebOps components?  That's totally workable.  If that's the case, does it make sense to collapse all the components under WebOps into one?

	Dylan Hardison [:dylan] (he/him) Assignee
	Comment 5 • 10 years ago

Currently, as it at the Product level rather than the component level, it would have to be presented for everything in the Infrastructure & Operations product -- which is more than just WebOps. Short of moving the WebOps components to a new product (which I am not in favor of) discussing bug 1037663 is the best idea.

	Dylan Hardison [:dylan] (he/him) Assignee
	Comment 6 • 10 years ago

The form is up on https://bugzilla-dev.allizom.org/form.webops.request.

Note: The WebOps: Request component is new, so I will need some more information to create it (I created it for bugzilla-dev with bogus description, etc).
The info I need is outlined in this wiki section: https://wiki.mozilla.org/BMO/Requesting_Changes#Components

	Mark Côté [:mcote]
	Comment 7 • 10 years ago

Is this not needed anymore, or is this just not useful until we implement bug 1037663?

	Sean Rich [:srich] Reporter
	Comment 8 • 10 years ago

Mark/Dylan - apologies for not following up on this sooner.  Let's block it on 1037663.  I think that until we can make this the "one to rule them all", this form won't be used.  I don't want to add cruft.  Meanwhile, we'll work with top requestors to give us the relevant info we need in existing components.  We'll also work to clean up components.

	Mark Côté [:mcote]
	Comment 9 • 10 years ago

Bug 1037663 has landed and been deployed.  Any forms for particular components are listed at the top of the "Enter bug" page for the given product (e.g. https://bugzilla.mozilla.org/enter_bug.cgi?product=mozilla.org).

Sean, shall we go ahead with your WebOps form?

	Sean Rich [:srich] Reporter
	Comment 10 • 10 years ago

Mark - thanks for the follow up.  Give me time to get the communications out around this change and then yes, let's make the change.  Question: As people start to use the new form and feedback comes in, does your team have the bandwidth to support tweaking the form as requested?

	Mark Côté [:mcote]
	Comment 11 • 10 years ago

Yes, we should be fix minor issues fairly quickly.  If it's something larger, it might take a week or two, but we'll try to work it in.

Also note that we generally push changes to BMO on a weekly basis, usually Monday night (ET/PT), including any form changes.

	Dylan Hardison [:dylan] (he/him) Assignee
	Comment 12 • 10 years ago

It's been so long since I touched this. 
I'll be adding it to the, err, suggested forms list (custom_forms.none.tmpl) and it should be good to go.

	Dylan Hardison [:dylan] (he/him) Assignee
	Comment 13 • 10 years ago

Finally in for review.

This uses the custom_forms.none.tmpl thing to register this form with the specified component.

	David Lawrence [:dkl]
	Comment 14 • 10 years ago

Comment on attachment 8538106 [details] [diff] [review]
bug-1027903-v1.patch

Review of attachment 8538106 [details] [diff] [review]:
-----------------------------------------------------------------

::: extensions/BMO/template/en/default/bug/create/create-webops-request.html.tmpl
@@ +18,5 @@
> +}
> +#webops .field_label {
> +  font-weight: bold;
> +}
> +#webops .field_desc {

Not used

@@ +21,5 @@
> +}
> +#webops .field_desc {
> +  padding-bottom: 3px;
> +}
> +#webops .field_desc,

Not used

@@ +37,5 @@
> +#webops textarea {
> +  font-family: inherit;
> +  font-size: inherit;
> +}
> +#webops em {

Not used

@@ +85,5 @@
> +   style = inline_style
> +   javascript = inline_javascript
> +   javascript_urls = [ 'extensions/BMO/web/js/form_validate.js',
> +                       'js/field.js', 'js/util.js' ]
> +   yui = [ "autocomplete", "calendar", "selector" ]

"calendar" not used

@@ +156,5 @@
> +  <div class="form_section">
> +    <label for="cc" class="field_label">CC</label><br>
> +    [% INCLUDE global/userselect.html.tmpl
> +      id       => "responsible_engineer"
> +      name     => "responsible_engineer"

Change the name and id to "cc", otherwise this value is not stored anywhere.

@@ +159,5 @@
> +      id       => "responsible_engineer"
> +      name     => "responsible_engineer"
> +      value    => ""
> +      size     => 80
> +      classes  => ["bz_userfield"]

add multiple => 5 to allow more than one user to be entered separated by commas.

	Dylan Hardison [:dylan] (he/him) Assignee
	Comment 15 • 10 years ago

I realized all the forms I've done are doing more work than they need to --
I thought the for= attribute of a label referenced the form element name=, rather than id= (durp)
so I use the CSS query bit from YUI to do a rather expensive search to map them to ids. I guess I can fix the other forms i've done eventually. XD

Revised with changes from review.

	David Lawrence [:dkl]
	Comment 16 • 10 years ago

Comment on attachment 8540736 [details] [diff] [review]
bug-1027903-v2.patch

Review of attachment 8540736 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good. r=dkl

	Dylan Hardison [:dylan] (he/him) Assignee
	Comment 17 • 10 years ago

To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
   89e09eb..66a9fc9  master -> master

	:glob ✱
	Comment 18 • 10 years ago

this form is now live at https://bugzilla.mozilla.org/form.webops.request