[TIMOB-17358] Android: Refactor notifications
GitHub Issue | n/a |
---|---|
Type | Bug |
Priority | High |
Status | Closed |
Resolution | Fixed |
Resolution Date | 2014-11-12T18:30:25.000+0000 |
Affected Version/s | n/a |
Fix Version/s | Release 4.0.0 |
Components | Android |
Labels | android-l |
Reporter | Ingo Muschenetz |
Assignee | Hieu Pham |
Created | 2014-07-22T16:23:47.000+0000 |
Updated | 2015-03-12T23:59:33.000+0000 |
Description
Our current implementation is deprecated, and likely to stop working with Android L.
Putting a SWAG of 2 weeks. We may want to consider adding some of the new features into notifications as well. We will also want to confirm that the new approach will work with Android Wear devices.
* Set up Eclipse * Set up Android studio * Update Studio * Update CLI * Refactoring notification
This requires an update to the AppCompat library which is not at GA status yet. Thus, we will be merging this to the master branch (not 3.4.0). Once Android L is closer to release and the proper components are available, we can slot this in. My goal is this would appear on the 3.4.X branch as soon as is feasible, and likely ship with 3.4.1.
master PR: https://github.com/appcelerator/titanium_mobile/pull/6247
Confirmed, we can merge this into 3.4.2.
Test case for new features of notification:
1. Send notification 1 first, then send notification 2. 2. Since notification 1 has higher priority than 2, 1 will be on top of 2. If you remove priority of notification 1, then repeat step 1, notification 2 will be on top of notification 1.
3.4.X PR: https://github.com/appcelerator/titanium_mobile/pull/6346
Verified the fix and support for notifications for android 5.0. Closing. Environment: Appc Studio : 4.0.0.201503062102 Ti SDK : 4.0.0.v20150312095013 CLI : 3.5.0-dev Alloy : 1.5.1 MAC Yosemite : 10.10.2 Nexus 5 - Android 5.0