Feature: Add field for Partner Engineer to be cc'ed on review

RESOLVED FIXED in 2013-05-02

Status

Marketplace
Reviewer Tools
--
major
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Harald, Assigned: robhudson)

Tracking

2013-05-02
Points:
---

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Marketplace currently allows a BD contact to be cc'ed on review result. Please add a field for Partner Engineering contact.

If a partner engineer is already in contact with the partners engineering team, it would make responses and hands-on help much faster when he gets notified with review feedback and can add additional notes.

This keeps up the energy on the partner side, keeps partner engineers in the loop and reduces forwarded emails. It also adds a new feedback-loop between partner engineering and the review team to improve and extend the provided feedback.

The combination of BD shouting encouragement and a partner engineer giving additional technical notes should tighten the review-resubmission loop.

It's a simple change and can be used versatility. A simple first step would be to allow a list of people for BD contact.

I'd love to hear thoughts! Do you see any drawbacks?
(Reporter)

Comment 1

5 years ago
Thinking about it, having one field for CCs that holds many user references would be better than adding fields for specific roles; if that's possible.

E.g. Evangelism might also want to be cc'ed for partners that they are coaching.
Just make the current contact field support a comma separated list.  (It has the bonus of requiring zero UX work.)
(Reporter)

Comment 3

5 years ago
(In reply to Andrew Williamson [:eviljeff] from comment #2)
> Just make the current contact field support a comma separated list.  (It has
> the bonus of requiring zero UX work.)

Agreed, that seems to be the simplest solution, low-impact and low-risk.
(In reply to Harald Kirschner :digitarald from comment #3)
> (In reply to Andrew Williamson [:eviljeff] from comment #2)
> > Just make the current contact field support a comma separated list.  (It has
> > the bonus of requiring zero UX work.)
> 
> Agreed, that seems to be the simplest solution, low-impact and low-risk.

Also agree. Can we move forward on this asap? I'm planning to use this to help Partner Engineering track and proactively respond to the preload app bugs that are being reported by OEM's and Carriers.
This seems like an easy and good change to me. Adding Bram who owns the design for this to get his input. Do we currently have a label for the cc field? If so, would we need to change that?
Flags: needinfo?(bram)
The label is 'Mozilla Contact'; the tooltip beside it is 'An e-mail address of this app's Mozilla Contact that will be CC'd on any reviewer emails'.  

Change to 'Mozilla Contact(s)' and 'A comma-separated list of e-mail addresses of this app's Mozilla Contact(s) that will be CC'd on any reviewer emails'.
I agree with Andrew’s suggestion on comment 2 and comment 6. Textbox that supports comma-separated lists is the easiest solution.

Can I get the screenshot or the URL of this text box, though, for a sanity check?
Flags: needinfo?(bram)
Created attachment 741575 [details]
screenshot of Admin Setting section, showing Mozilla contact field
(In reply to Andrew Williamson [:eviljeff] from comment #8)
> Created attachment 741575 [details]
> screenshot of Admin Setting section, showing Mozilla contact field

The tooltip you proposed on Comment 6 sounds fine. Additionally, let’s add a caption text (similar to the “264px x 184px” caption) below the “Mozilla Contact” heading that says:

Long version:
“You can enter more than one email address by separating each address with a comma”

Short version (if the long version breaks the layout too much):
“Enter email addresses, separated by commas”
When can we expect this to be implemented? As mentioned above this will be extremely useful in tracking/supporting Partner Engineering efforts. We have a hard deadline coming up 5/10 for content so the sooner the better :)
We'll have a triage meeting next week and try to get it into this quarter.
(Assignee)

Updated

5 years ago
Assignee: nobody → robhudson.mozbugs
Target Milestone: --- → 2013-05-02
(Assignee)

Comment 12

5 years ago
https://github.com/mozilla/zamboni/commit/22f10b8
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.