fx-private-relay-add-on icon indicating copy to clipboard operation
fx-private-relay-add-on copied to clipboard

Add Report Issue Panel

Open codemist opened this issue 3 years ago • 4 comments
trafficstars

This adds the Report issues feature on the add on where users can submit reports directly from the site where they are experiencing issues.

Inputs accepted:

  • Accepts non-formatted URLs: eg. test.com, test.co.uk, jane.app, www.jane.app, 見.香港
  • Disables submit button if the URL field is empty / does not contain a valid URL

Demo:

https://user-images.githubusercontent.com/13066134/196756228-178ae784-69f1-48b8-bd8a-98db7dd13283.mov

codemist avatar Jul 25 '22 20:07 codemist

@say-yawn Added you as a reviewer just in case you wanted to spot check.

codemist avatar Oct 14 '22 16:10 codemist

@maxxcrawford asked:

Is there any way for us to parse if the user reported from Chrome vs Firefox?

I could see two ways:

  1. The Add-on could report that it was from the Chrome store or Firefox Add-ons. Is this information available in the add-on?
  2. The backend could look at the User-Agent header. Looking at /metrics-event, we get the user agent, but we'll need to do additional parsing (and probably bring in a new library) to turn this into "Chrome", "Firefox", and "Something else"

jwhitlock avatar Oct 24 '22 17:10 jwhitlock

The add-on has access to runtime.getBrowserInfo and runtime.getPlatformInfo, but I think the backend will need an update to accept that data.

Vinnl avatar Oct 24 '22 18:10 Vinnl

@jwhitlock Se Yeon mentioned that ideally we should collect that data on the frontend and pass it. https://mozilla-hub.atlassian.net/jira/software/c/projects/MPP/boards/198?modal=detail&selectedIssue=MPP-2478

codemist avatar Oct 24 '22 19:10 codemist

@maxxcrawford Thanks for catching the errors in popup.html, that was a complete oversight. The differences now should be more clear, I reformatted the entire file with prettier and I thought that was what was causing the huge change and thought little of it. It should not have any bad rebase artifacts now.

codemist avatar Nov 10 '22 15:11 codemist