RestComm / restcomm-android-sdk

Android Mobile SDK to easily integrate communication features (WebRTC, messaging, presence, voice, video, screensharing) based on RestComm into native Mobile Applications
http://www.restcomm.com/
GNU Affero General Public License v3.0
153 stars 109 forks source link

Testing session issues for Push Notifications #786

Closed atsakiridis closed 6 years ago

atsakiridis commented 6 years ago

Generally, registerForPush() seems to be called even in cases where nothing changed, like leaving and re-entering App (check at the end of this log for such a case). This might cause confusion as we’ll be getting callbacks at times when user did nothing and there was not FCM token update. Also it makes the App take longer time to connect in those cases. Can we make sure that it is only called when something has actually changed either through initialize() or updateParams(), or if FCM token has been updated?