995037 - Migration logic to convert Notes (master) data format to Notes+ (phase2)

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Notes, defect)

Description

[Yan Or]

This will allow us to upgrade the Notes app to the latest Notes+ code.

Comment 1

[Yan Or]

Comparing data formats between Notes and Notes+, here are the transformations we need:

1. User object - none required
2. Notebooks - none required
3. Notes without resources
     - convert data_text to data_content (enml format)
     - add data_active:true
     - add data_resources:[]
4. Notes with resources
     - in addition to #3, add <en-media> node to end of data_content for each resource
     - add resource entry to data_resources for each resource
     - convert base64 encoded resource data to ArrayBuffer and generate appropriate hash
5. All object ids remain the same

Comment 2

[Yan Or]

Matias/Ran, please provide feedback.  Thanks.

Comment 3

[Ran Ben Aharon [:ranbena] EverythingMe]

Leaving the review for Matias

Comment 4

[Yan Or]

Attachment: Pull request

Comment 5

[Matias Crivolotti]

Comment on attachment 8407633 [details] [review]
Pull request

Yan, i went over the code, I couldn't test it on a device but it looks good to me...
If you have a way to test it on a device let me know how and i'll test it.

Good job.

Comment 6

[Yan Or]

Matias,

To test the migration:

1. Install the master branch build to your device
2. Create some notes
3. Update the app with the latest phase2 + this batch to your device
4. Data will be migrated when you open a note or when you do the first sync

I use App Manager to install and update the app.

Comment 7

[Ryan VanderMeulen [:RyanVM]]

Master: https://github.com/mozilla-b2g/notes/commit/ced0f7e5d2524f706211585750a19e7063babccb