search

ibm-bluemix-mobile-services / bms-clientsdk-android-push

Android Push notifications SDK for IBM Cloud Mobile Services
Apache License 2.0
10 stars 12 forks source link

Android O Push Notification #88

Open sairajkishore opened 7 years ago

sairajkishore commented 7 years ago

In android O am not able to get push notification.should we need to change notification builder?

AnanthaKrish commented 7 years ago

@sairajkishore It should work. I am using an Android O device for testing, it works. To get Android O based push notifications features we need to change the Notification Builder. We will release that soon.

sairajkishore commented 7 years ago

Also could you please update 1.Sample App for Android O support. 2.Also Library for Android O support which includes FCM libraries.

AnanthaKrish commented 7 years ago

@sairajkishore We will do that during the new sdk release

sairajkishore commented 7 years ago

Any idea when it could be in and around? as i just want to plan for my next release?

AnanthaKrish commented 7 years ago

@sairajkishore Do you want the specific Android O features to be available in SDK ?

sairajkishore commented 7 years ago

As Notification Manager changes is required to support Android O and Notification stuff is a part of SDK ,so we need updated SDK.

AnanthaKrish commented 7 years ago

If you want to try , please try to extend the com.ibm.mobilefirstplatform.clientsdk.android.push.api.MFPPushIntentService.java file

AnanthaKrish commented 7 years ago

@sairajkishore Without any modification in the current code Android O devices will get notifications. But this will not have any android O features. Are you able to get notifications in Android O ?

sairajkishore commented 7 years ago

If the preconditions are Build project with android O and target SDK is O, installed on android O device we are getting notification event but Notification manager does not show notification popup, that is the existing issue am facing.Earlier build for target 7 on O devices notification used to come.

in order to get this we need to set channel for Notification manager in O also fire base library also should be updated if we refer O documentation.So modifications are required definitely.

AnanthaKrish commented 7 years ago

@sairajkishore Yes I know that . I will let you know once its done.

sairajkishore commented 7 years ago

Also this is one of the crash log which i recently got.

Fatal Exception: java.lang.ArrayIndexOutOfBoundsException: length=1; index=6 at com.ibm.mobilefirstplatform.clientsdk.android.push.api.MFPPush.getInstance(Unknown Source) at com.ibm.mobilefirstplatform.clientsdk.android.push.api.MFPPushIntentService.setAppForeground(Unknown Source) at com.google.firebase.messaging.FirebaseMessagingService.handleIntent(Unknown Source) at com.google.firebase.iid.zzc.run(Unknown Source) at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1113) at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:588) at java.lang.Thread.run(Thread.java:818)