Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Remove Page Shot experiment migration code! #3844

Closed
jaredhirsch opened this issue Nov 27, 2017 · 0 comments
Closed

Remove Page Shot experiment migration code! #3844

jaredhirsch opened this issue Nov 27, 2017 · 0 comments
Assignees
@jaredhirsch
Labels
code quality Not a bug, targeted for fixing in 2018 code Code quality issue, that does not have any outward effect on the product firefox Something that has to ship in the add-on in Firefox
Milestone
Sprint 6 (60-1) 👨🏼‍🚀

Comments

@jaredhirsch
Copy link
Member

The Page Shot experiment is over, the service has been shut down.

Can we clear out the migration-related code yet?

Inspired by the first bit of feedback in https://bugzilla.mozilla.org/show_bug.cgi?id=1419148#c13
@johngruen johngruen added this to the Sprint 5 (60-0) 🔘 milestone Nov 28, 2017
@johngruen johngruen changed the title Remove Page Shot experiment migration code? Remove Page Shot experiment migration code! Nov 28, 2017
@jaredhirsch jaredhirsch mentioned this issue Dec 2, 2017
Closed
17 tasks
@ianb ianb added code Code quality issue, that does not have any outward effect on the product firefox Something that has to ship in the add-on in Firefox labels Jan 10, 2018
@jaredhirsch jaredhirsch self-assigned this Jan 16, 2018
@ianb ianb added the code quality Not a bug, targeted for fixing in 2018 label Jan 17, 2018
@jaredhirsch jaredhirsch mentioned this issue Feb 1, 2018
Merged
chenba pushed a commit that referenced this issue Feb 1, 2018
@jaredhirsch
Fix #3844, remove pageshot migration code
102724c
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jaredhirsch jaredhirsch

Labels
code quality Not a bug, targeted for fixing in 2018 code Code quality issue, that does not have any outward effect on the product firefox Something that has to ship in the add-on in Firefox
Projects
None yet
Milestone
Sprint 6 (60-1) 👨🏼‍🚀
Development

No branches or pull requests
3 participants
@ianb @jaredhirsch @johngruen