change remoteTroubleshooting js code to use promises

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
4 years ago
2 years ago

People

(Reporter: willkg, Unassigned, Mentored)

Tracking

Details

(Whiteboard: u=dev c=codequality p= s=)

The code in question is here:

https://github.com/mozilla/fjord/blob/d3fe0fcb975f7a81209b5e7d05d7f87e743f1a83/fjord/feedback/static/js/remote.js

This library currently uses callbacks to deal with the async remote-troubleshooting event. This bug covers changing it from using callbacks to using promises.
For browsers that don't support promises, we can assume there's no remote-troubleshooting data to get and the code should handle that situation accordingly. There's no need for polyfills here.
This requires someone who knows enough JavaScript to know about callbacks and promises and can convert the existing code to use promises.

The wry twist is that this code requires https and a whitelisted domain name in order for the remote-troubleshooting code to kick off. That's really hard to set up, so you'll have to figure out another way to test it or test an equivalent version of it.

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
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
(Assignee)

Updated

2 years ago
Product: Input → Input Graveyard
You need to log in before you can comment on or make changes to this bug.