<div dir="ltr">ah, I thought it would be still make sense to update the version there :-)<div><br></div><div>That's not a blocker ;-) Just noticed </div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 23, 2015 at 6:49 PM, Daniel Passos <span dir="ltr"><<a href="mailto:dpassos@redhat.com" target="_blank">dpassos@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Because test is not part of the release.<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 23, 2015 at 12:52 PM, Matthias Wessendorf <span dir="ltr"><<a href="mailto:matzew@apache.org" target="_blank">matzew@apache.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">this pom.xml, on the 2.2.0 tag has incorrect version nr: <a href="https://github.com/aerogear/aerogear-android-push/blob/2.2.0/aerogear-android-push-test/pom.xml#L29" target="_blank">https://github.com/aerogear/aerogear-android-push/blob/2.2.0/aerogear-android-push-test/pom.xml#L29</a> </div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 19, 2015 at 12:53 AM, Daniel Passos <span dir="ltr"><<a href="mailto:dpassos@redhat.com" target="_blank">dpassos@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi,<br><br></div>I've staged[1] the Push library 2.2.0 again and send a PR for UnifiedPush helloworld[2] using it.<br><p style="margin:1.2em 0px!important">We’re planning to release it next Wednesday </p><span>
<p style="margin:1.2em 0px!important">Fell free to test it and let us know if you find any issues.</p></span>[1] <a href="https://repository.jboss.org/nexus/content/repositories/jboss_releases_staging_profile-5856/" target="_blank">https://repository.jboss.org/nexus/content/repositories/jboss_releases_staging_profile-5856/</a><br>[2] <a href="https://github.com/jboss-mobile/unified-push-helloworld/pull/15" target="_blank">https://github.com/jboss-mobile/unified-push-helloworld/pull/15</a><br><br></div><div class="gmail_extra"><div><div><br><div class="gmail_quote">On Wed, Jun 17, 2015 at 10:36 PM, Daniel Passos <span dir="ltr"><<a href="mailto:dpassos@redhat.com" target="_blank">dpassos@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hey Guys,<br><br>We are postpone it to fix the bug Matthias found. I just send a PR[1] to fix it.<br><br>[1] <a href="https://github.com/aerogear/aerogear-android-push/pull/38" target="_blank">https://github.com/aerogear/aerogear-android-push/pull/38</a><br></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 17, 2015 at 11:32 AM, Summers Pittman <span dir="ltr"><<a href="mailto:supittma@redhat.com" target="_blank">supittma@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div><br></div><div class="gmail_extra"><br><div class="gmail_quote"><span>On Wed, Jun 17, 2015 at 3:17 AM, Sebastien Blanc <span dir="ltr"><<a href="mailto:scm.blanc@gmail.com" target="_blank">scm.blanc@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">This fix is more a workaround since it was not needed with the previous versions. so we need to be sure to check if :</div></blockquote></span><div>No. The hello world app is broken. </div><span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><br><div>- This fix was needed because of other commits on the helloworld </div></div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><div>- This fix was needed because of the new version of the library</div><div>- This fix was needed because of a change from GCM server</div><div>- This fix was needed because of a mix/combination of all the previous potential causes</div></div></div></blockquote><div><br></div></span><div>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. <br></div><div><br></div><div>The application work by happy coincidence. It wasn't until matz tested on a hellishly obsolete device that the issue appeared. ;)</div><div><div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div></div><div>Passos have you been able to test the lib with Aerodoc ? </div><div><br></div><div>Sebi</div><div><br></div><div><br></div><div><br></div></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 16, 2015 at 4:50 PM, Daniel Passos <span dir="ltr"><<a href="mailto:dpassos@redhat.com" target="_blank">dpassos@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">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]<br><br>[1] <a href="https://issues.jboss.org/browse/AGDROID-425?focusedCommentId=13079805&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13079805" target="_blank">https://issues.jboss.org/browse/AGDROID-425?focusedCommentId=13079805&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13079805</a><br>[2] <a href="https://github.com/jboss-mobile/unified-push-helloworld/pull/14" target="_blank">https://github.com/jboss-mobile/unified-push-helloworld/pull/14</a><br></div><div class="gmail_extra"><div><div><br><div class="gmail_quote">On Mon, Jun 15, 2015 at 9:42 AM, Matthias Wessendorf <span dir="ltr"><<a href="mailto:matzew@apache.org" target="_blank">matzew@apache.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">I noticed a little bug w/ the current libray<div><br></div><div><a href="https://issues.jboss.org/browse/AGDROID-425" target="_blank">https://issues.jboss.org/browse/AGDROID-425</a><br></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div>On Mon, Jun 15, 2015 at 2:42 AM, Daniel Passos <span dir="ltr"><<a href="mailto:dpassos@redhat.com" target="_blank">dpassos@redhat.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div><div dir="ltr"><div><p style="margin:1.2em 0px!important">Hey Everyone,</p>
<p style="margin:1.2em 0px!important">We are ready to release AeroGear Android Push 2.2.0, we staged[1] it on Nexus. </p>
<p style="margin:1.2em 0px!important">What is new in this version?</p>
<ul style="margin:1.2em 0px;padding-left:2em">
<li style="margin:0.5em 0px"><a href="https://issues.jboss.org/browse/AGDROID-422" target="_blank">AGDROID-422</a> - Add support for push-config.json for Android Push SDK</li>
<li style="margin:0.5em 0px"><a href="https://issues.jboss.org/browse/AGDROID-423" target="_blank">AGDROID-423</a> - Android SDK hook for notifying the server if app has been opened due to a push</li></ul><ul style="margin:1.2em 0px;padding-left:2em">
</ul>
<p style="margin:1.2em 0px!important">We’re planning to release it next Thursday. </p>
<p style="margin:1.2em 0px!important">Fell free to test it and let us know if you find any issues.</p>
<p style="margin:1.2em 0px!important">[1] <a href="https://repository.jboss.org/nexus/content/repositories/jboss_releases_staging_profile-5783/" target="_blank">https://repository.jboss.org/nexus/content/repositories/jboss_releases_staging_profile-5783/</a></p>
<div title="MDH:PGRpdj48c3Bhbj5IZXkgRXZlcnlvbmUsPC9zcGFuPjxicj48YnI+PHNwYW4+V2UgYXJlIHJlYWR5
IHRvIDxzcGFuPnJlbGVhc2U8L3NwYW4+IDxzcGFuPkFlcm9HZWFyPC9zcGFuPiA8c3Bhbj5BbmRy
b2lkPC9zcGFuPiBQdXNoIDIuMi4wLCB3ZSBzdGFnZWRbMV0gaXQgb24gTmV4dXMuIDxicj48YnI+
PC9zcGFuPjwvZGl2PjxkaXY+PHNwYW4+V2hhdCBpcyBuZXcgaW4gdGhpcyB2ZXJzaW9uPzxicj48
L3NwYW4+PC9kaXY+PGRpdj48c3Bhbj48YnI+KiBbPC9zcGFuPjxzcGFuPjxzcGFuPkFHRFJPSUQt
NDIyXSg8L3NwYW4+PGEgaHJlZj0iaHR0cHM6Ly9pc3N1ZXMuamJvc3Mub3JnL2Jyb3dzZS9BR0RS
T0lELTQyMiIgdGFyZ2V0PSJfYmxhbmsiPmh0dHBzOi8vaXNzdWVzLjx3YnI+amJvc3Mub3JnL2Jy
b3dzZS9BR0RST0lELTQyMjwvYT4pIC0gQWRkIHN1cHBvcnQgZm9yIHB1c2gtY29uZmlnLmpzb24g
Zm9yIEFuZHJvaWQgUHVzaCBTREs8YnI+KiBbQUdEUk9JRC00MjNdKDxhIGhyZWY9Imh0dHBzOi8v
aXNzdWVzLmpib3NzLm9yZy9icm93c2UvQUdEUk9JRC00MjMiIHRhcmdldD0iX2JsYW5rIj5odHRw
czovL2lzc3Vlcy48d2JyPmpib3NzLm9yZy9icm93c2UvQUdEUk9JRC00MjM8L2E+KSAtIEFuZHJv
aWQgU0RLIGhvb2sgZm9yIG5vdGlmeWluZyB0aGUgc2VydmVyIGlmIGFwcCBoYXMgYmVlbiBvcGVu
ZWQgZHVlIHRvIGEgcHVzaDxicj48L3NwYW4+PC9kaXY+PHNwYW4+PC9zcGFuPjxkaXY+PHNwYW4+
PGJyPldlJ3JlIHBsYW5uaW5nIHRvIDxzcGFuPnJlbGVhc2U8L3NwYW4+IGl0IG5leHQgVGh1cnNk
YXkuIDxicj48YnI+RmVsbCBmcmVlIHRvIHRlc3QgaXQgYW5kIGxldCB1cyBrbm93IGlmIHlvdSBm
aW5kIGFueSBpc3N1ZXMuPC9zcGFuPjxicj48YnI+PHNwYW4+WzFdIDwvc3Bhbj48YSBocmVmPSJo
dHRwczovL3JlcG9zaXRvcnkuamJvc3Mub3JnL25leHVzL2NvbnRlbnQvcmVwb3NpdG9yaWVzL2pi
b3NzX3JlbGVhc2VzX3N0YWdpbmdfcHJvZmlsZS01NzgzLyIgdGFyZ2V0PSJfYmxhbmsiPmh0dHBz
Oi8vcmVwb3NpdG9yeS5qYm9zcy5vcmcvPHdicj5uZXh1cy9jb250ZW50L3JlcG9zaXRvcmllcy88
d2JyPmpib3NzX3JlbGVhc2VzX3N0YWdpbmdfPHdicj5wcm9maWxlLTU3ODMvPC9hPjxiciBjbGVh
cj0iYWxsIj48YnI+PC9kaXY+" style="min-height:0px;width:0px;max-height:0px;max-width:0px;overflow:hidden;font-size:0em;padding:0px;margin:0px"></div></div><span><font color="#888888"><div>-- <br><div><div dir="ltr">-- Passos<br></div></div>
</div></font></span></div>
<br></div></div>_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org" target="_blank">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><span><font color="#888888"><br></font></span></blockquote></div><span><font color="#888888"><br><br clear="all"><div><br></div>-- <br><div>Matthias Wessendorf <br><br>blog: <a href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a></div>
</font></span></div>
<br>_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org" target="_blank">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br></blockquote></div><br><br clear="all"><br>-- <br></div></div><span><font color="#888888"><div><div dir="ltr">-- Passos<br></div></div>
</font></span></div>
<br>_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org" target="_blank">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br></blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org" target="_blank">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br></blockquote></div></div></div><br></div></div>
<br>_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org" target="_blank">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br></blockquote></div><br><br clear="all"><br>-- <br><div><div dir="ltr">-- Passos<br></div></div>
</div>
</div></div></blockquote></div><br><br clear="all"><br></div></div><span><font color="#888888">-- <br><div><div dir="ltr">-- Passos<br></div></div>
</font></span></div>
<br>_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org" target="_blank">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Matthias Wessendorf <br><br>blog: <a href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a></div>
</div>
</div></div><br>_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org" target="_blank">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br></blockquote></div><br><br clear="all"><br>-- <br><div><div dir="ltr">-- Passos<br></div></div>
</div>
</div></div><br>_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Matthias Wessendorf <br><br>blog: <a href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a></div>
</div>