1612982 - [meta] Remove server code from Screenshots

Status: NEW

(Firefox :: Screenshots, task, P3)

Description

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

Now that the Screenshots server has been removed and shutdown, we can clean up all the server-related code from the extension. Here's a terse, but still useful, checklist from https://github.com/mozilla-services/screenshots/issues/5352:

• The authentication code
• All deviceId-related code
• Removing that from shot.js
• Removing the actual save code
• Removing any My Shots code
• Removing any actual sending of events via sendEvent (sending some of those events to Telemetry will be another task, for this bug we should simply collect but not send events)
• Removing WebExperiment code that is no longer used after this
• Removing any related build settings (maybe only relevant for github)
  This does not have to include removing Sentry submission, though we will want to debate that separately.

FWIW, I don't see value in continuing to send errors to Sentry, since no one's triaging the Sentry reports.

Comment 2

[Sam Foster [:sfoster] (he/him)]

A single patch to remove all this would be a bit overwhelming, lets break it out into a few steps to remove completely orphaned code, and to untangle the my shots feature from the current working code.

Comment 3

[u682976]

Can I be assigned to these issues?

Comment 4

[Emma Malysz]

Hi Dawit, please start with one of the dependencies for now