[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.