[ALOY-950] 'localStorage' adapter for Alloy Models should be deprecated and no longer used

GitHub Issue	n/a
Type	Improvement
Priority	Low
Status	Closed
Resolution	Fixed
Resolution Date	2014-07-31T15:26:07.000+0000
Affected Version/s	n/a
Fix Version/s	Alloy 1.5.0
Components	Models
Labels	n/a
Reporter	Federico Casali
Assignee	Feon Sua Xin Miao
Created	2014-02-06T00:16:50.000+0000
Updated	2014-08-18T19:09:39.000+0000

Description

Improvement description

We should deprecate the usage of the localStorage adapter for Alloy Models. It could create confusion as only supports MobileWeb and Tizen and is outdated by the 'properties' adapter (that supports all platforms). See also http://docs.appcelerator.com/titanium/latest/#!/guide/Alloy_Sync_Adapters_and_Migrations-section-36739597_AlloySyncAdaptersandMigrations-Ready-MadeSyncAdapters Will need updates to documentation that includes sample code to use Properties rather than localStorage. Tizen support has already been removed, so this will focus on only MobileWeb. Make sure the change is communicated in the release notes.

Comments

Feon Sua Xin Miao 2014-07-31

PR: https://github.com/appcelerator/alloy/pull/496 Removed sample usage and also created tickets to: TIDOC to update release notes TIDOC to update documentation ALOY to remove the feature in next release
Tim Poulsen 2014-07-31

PR merged
Federico Casali 2014-08-18

localStorage sample correctly removed. (for MobileWeb, 'properties' adapter is available here: https://github.com/appcelerator/alloy/tree/master/test/apps/models/properties) Closing.

JSON Source