1109920 - Remove collection migration code from FTU

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::First Time Experience, defect)

Description

[Kevin Grandon :kgrandon]

It is no longer needed now that we don't need to support migrations from 1.4 -> 2.2+.

Comment 1

[GitHub Autolander Bot]

Attachment: [PullReq] KevinGrandon:bug_1109920_remove_ftu_migration to mozilla-b2g:master

Comment 2

[Kevin Grandon :kgrandon]

Comment on attachment 8534642 [details] [review]
[PullReq] KevinGrandon:bug_1109920_remove_ftu_migration to mozilla-b2g:master

Sam - could you give this a review when you have time? Thanks!

Comment 3

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

(In reply to Kevin Grandon :kgrandon from comment #2)
> Comment on attachment 8534642 [details] [review]
> [PullReq] KevinGrandon:bug_1109920_remove_ftu_migration to mozilla-b2g:master
> 
> Sam - could you give this a review when you have time? Thanks!

The patch itself looks fine, but what is the decision that makes this a requirement? Where/when did we decide not to support migrations from 1.4? For the purposes of testing, what are the expected results?

Comment 4

[Kevin Grandon :kgrandon]

We have decided that we will only support migrations across 2 versions. E.g., 1.3 -> 2.0, or 1.4 -> 2.1.

The *lowest* possible version number to migrate to 2.2 is 2.0, so for that migration you do not need the FTU migration step.

For the purposes of testing, I suppose you could flash a device and make sure smart collections show up. Previously this would've migrated user-created smart collections to the home screen. We no longer need to do that.

Comment 5

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

Comment on attachment 8534642 [details] [review]
[PullReq] KevinGrandon:bug_1109920_remove_ftu_migration to mozilla-b2g:master

Thanks for doing this. Honestly I'm not clear why this was in the Tutorial code to begin with and am glad to see it gone :) 

QA note: this patch removes legacy migration code for 1.x smart collections that is no longer supported or needed (users will upgrade to 2.0 then 2.x). The automated tests give us the best coverage. If you want to manually verify, just flash/reset and step through the FTU tutorial to confirm there are no new blockages or exceptions.

Comment 6

[Kevin Grandon :kgrandon]

In master: https://github.com/mozilla-b2g/gaia/commit/a2e8446022cbad0ec2d3c3561619820433e45181