Closed Bug 1128693 Opened 10 years ago Closed 8 years ago

fix browser data capture code in generic feedback form

Categories

(Input Graveyard :: Code Quality, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: willkg, Unassigned, Mentored)

Details

(Whiteboard: u=user c=feedback p= s=)

https://bugzilla.mozilla.org/show_bug.cgi?id=1119813#c3 The way I wrote the code to capture the browser data, I have it reacting to events on the checkbox. That's kind of silly. This bug covers rewriting that code so that it continues to ask the browser for browser data when the form loads, but it copies the data into the appropriate place to submit when the user clicks the Submit button.
This requires knowledge of JavaScript. The code in question is here: https://github.com/mozilla/fjord/blob/14e7ac48a09d7b3a58ec18559e57f7d828e36d98/fjord/feedback/static/js/generic_feedback_dev.js#L151 However, it's likely I'll have merged the generic_feedback_dev.js file into generic_feedback.js. You'll need to make the changes to generic_feedback.js. The wry twist is that it's difficult to get everything working such that the code kicks off and there aren't any instructions for it. Without doing that, the data comes back as undefined because the code doesn't have permissions to access that API: https://github.com/mozilla/fjord/blob/14e7ac48a09d7b3a58ec18559e57f7d828e36d98/fjord/feedback/static/js/remote.js#L69 If you get far enough to do a PR, I can test it. Even so, marking this as a mentored bug. If you're interested in working on it, please read through our "Join this project" guide [1], complete the list of things you need to do before contributing and then add a comment to this bug asking to have it assigned to you. If you have any questions, please ask in the bug comments, on the mailing list or in the #input channel on irc.mozilla.org. [1] http://fjord.readthedocs.org/en/latest/welcome.html
Mentor: willkg
The Input service has been decommissioned (see bug 1315316) and has been replaced by a redirect to an external vendor (SurveyGizmo). I'm bulk WONTFIXing Input bugs that do not appear to be relevant anymore.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Product: Input → Input Graveyard
You need to log in before you can comment on or make changes to this bug.