[aerogear-dev] AeroGear Android Push 2.2.0

[Summers Pittman]

On Wed, Jun 17, 2015 at 3:17 AM, Sebastien Blanc <scm.blanc at gmail.com>
wrote:

> This fix is more a workaround since it was not needed with the previous
> versions.  so we need to be sure to check if :
>
No.  The hello world app is broken.

>
> - This fix was needed because of other commits on the helloworld
>
- This fix was needed because of the new version of the library
> - This fix was needed because of a change from GCM server
> - This fix was needed because of a mix/combination of all the previous
> potential causes
>

TL;DR: We never needed information from the PushRegistrar to handle
messages until metrics were added.  Now we do.  However, if the app is
garbage collected then the registration information is cleared from memory
(it is persisted on the device of course).  Because the notification
handler is invoked by android it bypasses the RegisterActivity.  When we
try to fetch the registration information to send the metric message it
isn't loaded and the NPE is thrown.  Previously had enabled the ability to
unregister the client from the message list screen we would have seen
similar results.

The application work by happy coincidence.  It wasn't until matz tested on
a hellishly obsolete device that the issue appeared. ;)



> Passos have you been able to test the lib with Aerodoc ?
>
> Sebi
>
>
>
>
> On Tue, Jun 16, 2015 at 4:50 PM, Daniel Passos <dpassos at redhat.com> wrote:
>
>> Seems the problem was in the app not in the library. For more details
>> please see my comment[1] in Jira and the fix in this PR[2]
>>
>> [1]
>> https://issues.jboss.org/browse/AGDROID-425?focusedCommentId=13079805&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13079805
>> [2] https://github.com/jboss-mobile/unified-push-helloworld/pull/14
>>
>> On Mon, Jun 15, 2015 at 9:42 AM, Matthias Wessendorf <matzew at apache.org>
>> wrote:
>>
>>> I noticed a little bug w/ the current libray
>>>
>>> https://issues.jboss.org/browse/AGDROID-425
>>>
>>>
>>>
>>> On Mon, Jun 15, 2015 at 2:42 AM, Daniel Passos <dpassos at redhat.com>
>>> wrote:
>>>
>>>> Hey Everyone,
>>>>
>>>> We are ready to release AeroGear Android Push 2.2.0, we staged[1] it on
>>>> Nexus.
>>>>
>>>> What is new in this version?
>>>>
>>>>    - AGDROID-422 <https://issues.jboss.org/browse/AGDROID-422> - Add
>>>>    support for push-config.json for Android Push SDK
>>>>    - AGDROID-423 <https://issues.jboss.org/browse/AGDROID-423> -
>>>>    Android SDK hook for notifying the server if app has been opened due to a
>>>>    push
>>>>
>>>>
>>>>
>>>> We’re planning to release it next Thursday.
>>>>
>>>> Fell free to test it and let us know if you find any issues.
>>>>
>>>> [1]
>>>> https://repository.jboss.org/nexus/content/repositories/jboss_releases_staging_profile-5783/
>>>> ​
>>>> --
>>>> -- Passos
>>>>
>>>> _______________________________________________
>>>> aerogear-dev mailing list
>>>> aerogear-dev at lists.jboss.org
>>>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>>>>
>>>
>>>
>>>
>>> --
>>> Matthias Wessendorf
>>>
>>> blog: http://matthiaswessendorf.wordpress.com/
>>> sessions: http://www.slideshare.net/mwessendorf
>>> twitter: http://twitter.com/mwessendorf
>>>
>>> _______________________________________________
>>> aerogear-dev mailing list
>>> aerogear-dev at lists.jboss.org
>>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>>>
>>
>>
>>
>> --
>> -- Passos
>>
>> _______________________________________________
>> aerogear-dev mailing list
>> aerogear-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>>
>
>
> _______________________________________________
> aerogear-dev mailing list
> aerogear-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/aerogear-dev/attachments/20150617/1770ed7e/attachment-0001.html