Closed
Bug 1104935
Opened 10 years ago
Closed 10 years ago
implement about:support section in generic feedback form
Categories
(Input Graveyard :: Submission, defect, P1)
Input Graveyard
Submission
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: willkg, Assigned: willkg)
References
Details
(Whiteboard: u=user c=feedback p=2 s=input.2014q4)
We need to write the code that asks the user for consent and then captures the about:support data to send to Input on submission.
This bug covers that.
Specifically:
1. add section to the details part of the generic feedback form asking the user to check the box if they consent to having their about:support data sent to Input along with their feedback and that this information is private
2. add some JS to parse the about:support data and put it in a form that's appropriate for submission to Input
3. implement the code in the feedback handling view to capture the about:support data and put it in the db
4. tests
5. smoke tests
Assignee | ||
Comment 1•10 years ago
|
||
For item 1, we need to get any text approved by Matej.
For item 3, because the api is whitelisted, we might have difficulties testing it in non-live environments.
Further, we have to wait to do this until we land the new generic feedback form which is bug #1099138.
Depends on: 1099138
Assignee | ||
Comment 2•10 years ago
|
||
Need to rip out the old generic form code before we can work on this.
Depends on: 1105512
Assignee | ||
Comment 3•10 years ago
|
||
I ripped out the old generic form code just now. It's just blocked on landing the models (bug #1104934).
I was thinking the text could be something like this:
(Optional) Check here to include your browser data
which will help us understand the problem. This data
will be kept private.
It needs word-smithing and approval from Matej.
Depends on: 1104934
Assignee | ||
Comment 4•10 years ago
|
||
Matt suggests:
Sad "(Optional) Check here to provide technical information about your browser. This will help us diagnose problems more quickly. This data will be kept private."
Happy "(Optional) Check here to provide technical information about your browser. This information helps us better understand our users and create better products. This data will be kept private.
Assignee | ||
Comment 5•10 years ago
|
||
In a PR: https://github.com/mozilla/fjord/pull/410
I also sent Matej an email asking for thoughts on the text.
Assignee | ||
Comment 6•10 years ago
|
||
Got the +1 from Matej via email. So we're all set in that department.
Comment 7•10 years ago
|
||
Great news! This is going to be a huge win for us and our users. Looking forward to getting this landed.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → willkg
Status: NEW → ASSIGNED
Whiteboard: u=user c=feedback p= s=input.2014q4 → u=user c=feedback p=2 s=input.2014q4
Assignee | ||
Comment 8•10 years ago
|
||
Landed in master in:
https://github.com/mozilla/fjord/commit/93d317bde556065d62566e3366d4c766196015b9
https://github.com/mozilla/fjord/commit/ad3a5cbba3bdb9981c8bc37de1577a58d7b63162
I'll push this on Monday.
Then we'll flick the switch so that en-US users see it and see how it goes. If we have problems, we can flick the switch back and then assess. Which reminds me we need a way to see how many people are opting in and how many are not.
After this is closed, we'll need a bug for making it universally available (i.e. not en-US only) which will block on a strings sync with Verbatim and translations being complete and in production.
Unfortunately, we don't have infrastructure for turning it on locale-by-locale. If we find ourselves in this situation often, then we should build that so we can roll things out more granularly.
Assignee | ||
Comment 9•10 years ago
|
||
Added metrics in PR: https://github.com/mozilla/fjord/pull/413
Landed in: https://github.com/mozilla/fjord/commit/cc54daf0d30b48f15077a9381a6070757b4c33d6
Assignee | ||
Comment 10•10 years ago
|
||
Pushed this to production, verified it's working and flipped the feedbackdev flag to "Everyone=True" so everyone can see it now.
I'm watching it in graphite now. One thing I just realized is that "optout" is split between "browser doesn't support it" and "user actually opted out". I should rename that to something more accurate.
Will close this later if things continue to look ok.
Assignee | ||
Comment 11•10 years ago
|
||
I saw an optin and an optout go by. I think everything is looking good so far. Marking as FIXED.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Product: Input → Input Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•