onOpened/onReceived events do not firing on android

[Ble3k]

Description:
Hi everybody. I'm having troubles with onOpened/onReceived events on android. Behavior is different on emulator and on a real device, but exact push message is delivered in both cases.

On emulator:

1. Foreground - onReceived event is invoked, onOpened is invoked only when push is tapped (correct)
2. Background active - none fired at all (incorrect)
3. Background inactive (app is closed) - onOpened is invoked only when push is tapped (correct)

On a real device:

1. Foreground - none fired at all (incorrect)
2. Background active - none fired at all (incorrect)
3. Background inactive (app is closed) - onOpened is invoked only when push is tapped (correct)

Environment

1. react-native-onesignal: "3.2.4"
2. react-native: "0.50.4"
3. compileSdkVersion: "27"
4. buildToolsVersion: "27.0.3"
5. gradle: "4.1"

Steps to Reproduce Issue:
Can't provide steps to reproduce due to my company's policy of confidence.

Anything else:
android/build.gradle

buildscript {
    repositories {
        jcenter()
        google()
    }
    dependencies {
        classpath 'com.android.tools.build:gradle:3.0.1'

        // NOTE: Do not place your application dependencies here; they belong
        // in the individual module build.gradle files
    }
}

allprojects {
    repositories {
        maven { url "https://jitpack.io" }    // used for MPAndroidChart
        mavenLocal()
        google()
        jcenter()
        maven {
            // All of React Native (JS, Obj-C sources, Android binaries) is installed from npm
            url "$rootDir/../node_modules/react-native/android"
        }
    }
}

android/app/build.gradle

apply plugin: 'com.onesignal.androidsdk.onesignal-gradle-plugin'
apply plugin: "com.android.application"

import com.android.build.OutputFile

buildscript {
    repositories {
        maven { url 'https://plugins.gradle.org/m2/' } // Gradle Plugin Portal 
    }
    dependencies {
        classpath 'gradle.plugin.com.onesignal:onesignal-gradle-plugin:[0.10.0, 0.99.99]'
    }
}

apply plugin: 'com.onesignal.androidsdk.onesignal-gradle-plugin'

/**
 * The react.gradle file registers a task for each build variant (e.g. bundleDebugJsAndAssets
 * and bundleReleaseJsAndAssets).
 * These basically call `react-native bundle` with the correct arguments during the Android build
 * cycle. By default, bundleDebugJsAndAssets is skipped, as in debug/dev mode we prefer to load the
 * bundle directly from the development server. Below you can see all the possible configurations
 * and their defaults. If you decide to add a configuration block, make sure to add it before the
 * `apply from: "../../node_modules/react-native/react.gradle"` line.
 *
 * project.ext.react = [
 *   // the name of the generated asset file containing your JS bundle
 *   bundleAssetName: "index.android.bundle",
 *
 *   // the entry file for bundle generation
 *   entryFile: "index.android.js",
 *
 *   // whether to bundle JS and assets in debug mode
 *   bundleInDebug: false,
 *
 *   // whether to bundle JS and assets in release mode
 *   bundleInRelease: true,
 *
 *   // whether to bundle JS and assets in another build variant (if configured).
 *   // See http://tools.android.com/tech-docs/new-build-system/user-guide#TOC-Build-Variants
 *   // The configuration property can be in the following formats
 *   //         'bundleIn${productFlavor}${buildType}'
 *   //         'bundleIn${buildType}'
 *   // bundleInFreeDebug: true,
 *   // bundleInPaidRelease: true,
 *   // bundleInBeta: true,
 *
 *   // whether to disable dev mode in custom build variants (by default only disabled in release)
 *   // for example: to disable dev mode in the staging build type (if configured)
 *   devDisabledInStaging: true,
 *   // The configuration property can be in the following formats
 *   //         'devDisabledIn${productFlavor}${buildType}'
 *   //         'devDisabledIn${buildType}'
 *
 *   // the root of your project, i.e. where "package.json" lives
 *   root: "../../",
 *
 *   // where to put the JS bundle asset in debug mode
 *   jsBundleDirDebug: "$buildDir/intermediates/assets/debug",
 *
 *   // where to put the JS bundle asset in release mode
 *   jsBundleDirRelease: "$buildDir/intermediates/assets/release",
 *
 *   // where to put drawable resources / React Native assets, e.g. the ones you use via
 *   // require('./image.png')), in debug mode
 *   resourcesDirDebug: "$buildDir/intermediates/res/merged/debug",
 *
 *   // where to put drawable resources / React Native assets, e.g. the ones you use via
 *   // require('./image.png')), in release mode
 *   resourcesDirRelease: "$buildDir/intermediates/res/merged/release",
 *
 *   // by default the gradle tasks are skipped if none of the JS files or assets change; this means
 *   // that we don't look at files in android/ or ios/ to determine whether the tasks are up to
 *   // date; if you have any other folders that you want to ignore for performance reasons (gradle
 *   // indexes the entire tree), add them here. Alternatively, if you have JS files in android/
 *   // for example, you might want to remove it from here.
 *   inputExcludes: ["android/**", "ios/**"],
 *
 *   // override which node gets called and with what additional arguments
 *   nodeExecutableAndArgs: ["node"],
 *
 *   // supply additional arguments to the packager
 *   extraPackagerArgs: []
 * ]
 */

project.ext.react = [
    cliPath: "node_modules/haul/bin/cli.js"
]

apply from: "../../node_modules/react-native/react.gradle"

/**
 * Set this to true to create two separate APKs instead of one:
 *   - An APK that only works on ARM devices
 *   - An APK that only works on x86 devices
 * The advantage is the size of the APK is reduced by about 4MB.
 * Upload all the APKs to the Play Store and people will download
 * the correct one based on the CPU architecture of their device.
 */
def enableSeparateBuildPerCPUArchitecture = false

/**
 * Run Proguard to shrink the Java bytecode in release builds.
 */
def enableProguardInReleaseBuilds = false

android {
    compileSdkVersion 27
    buildToolsVersion "27.0.3"

    defaultConfig {
        applicationId "com.binomysteryshopper"
        minSdkVersion 16
        targetSdkVersion 22
        versionCode 1
        versionName "1.0"
        vectorDrawables.useSupportLibrary = true
        ndk {
            abiFilters "armeabi-v7a", "x86"
        }
    }
    signingConfigs {
          release {
              if (project.hasProperty('MYAPP_RELEASE_STORE_FILE')) {
                  storeFile file(MYAPP_RELEASE_STORE_FILE)
                  storePassword MYAPP_RELEASE_STORE_PASSWORD
                  keyAlias MYAPP_RELEASE_KEY_ALIAS
                  keyPassword MYAPP_RELEASE_KEY_PASSWORD
              }
          }
      }
    splits {
        abi {
            reset()
            enable enableSeparateBuildPerCPUArchitecture
            universalApk false  // If true, also generate a universal APK
            include "armeabi-v7a", "x86"
        }
    }
    buildTypes {
        release {
            minifyEnabled enableProguardInReleaseBuilds
            proguardFiles getDefaultProguardFile("proguard-android.txt"), "proguard-rules.pro"
            signingConfig signingConfigs.release
        }
    }
    // applicationVariants are e.g. debug, release
    applicationVariants.all { variant ->
        variant.outputs.each { output ->
            // For each separate APK per architecture, set a unique version code as described here:
            // http://tools.android.com/tech-docs/new-build-system/user-guide/apk-splits
            def versionCodes = ["armeabi-v7a":1, "x86":2]
            def abi = output.getFilter(OutputFile.ABI)
            if (abi != null) {  // null for the universal-debug, universal-release variants
                output.versionCodeOverride =
                        versionCodes.get(abi) * 1048576 + defaultConfig.versionCode
            }
        }
    }
}

dependencies {
    compile (project(':react-native-onesignal')){
        exclude group: "com.google.android.gms"
    }
    compile ("com.google.android.gms:play-services-base:12.0.1") {
        force = true;
    }
    compile project(':react-native-sound')
    compile project(':react-native-audio')
    compile project(':react-native-fetch-blob')
    compile project(':react-native-fs')
    compile project(':react-native-geocoder')
    compile project(':react-native-image-crop-picker')
    compile project(':react-native-version-check')
    compile project(':react-native-charts-wrapper')
    compile project(':react-native-spinkit')
    compile project(':react-native-svg')
    compile project(':react-native-maps')
    compile fileTree(dir: "libs", include: ["*.jar"])
    compile "com.android.support:appcompat-v7:23.0.1"
    compile "com.facebook.react:react-native:+"  // From node_modules
}

// Run this once to be able to run the application with BUCK
// puts all compile dependencies into folder libs for BUCK to use
task copyDownloadableDepsToLibs(type: Copy) {
    from configurations.compile
    into 'libs'
}

android/app/src/main/java/com/binomysteryshopper/MainApplication.java

package com.binomysteryshopper;

import android.app.Application;

import com.facebook.react.ReactApplication;
import com.geektime.rnonesignalandroid.ReactNativeOneSignalPackage;
import com.zmxv.RNSound.RNSoundPackage;
import com.rnim.rn.audio.ReactNativeAudioPackage;
import com.RNFetchBlob.RNFetchBlobPackage;
import com.rnfs.RNFSPackage;
import com.reactnative.ivpusic.imagepicker.PickerPackage;
import io.xogus.reactnative.versioncheck.RNVersionCheckPackage;
import com.github.wuxudong.rncharts.MPAndroidChartPackage;
import com.react.rnspinkit.RNSpinkitPackage;
import com.horcrux.svg.SvgPackage;
import com.facebook.react.ReactNativeHost;
import com.facebook.react.ReactPackage;
import com.facebook.react.shell.MainReactPackage;
import com.devfd.RNGeocoder.RNGeocoderPackage;
import com.facebook.soloader.SoLoader;
import com.airbnb.android.react.maps.MapsPackage;

import java.util.Arrays;
import java.util.List;

public class MainApplication extends Application implements ReactApplication {

  private final ReactNativeHost mReactNativeHost = new ReactNativeHost(this) {
    @Override
    public boolean getUseDeveloperSupport() {
      return BuildConfig.DEBUG;
    }

    @Override
    protected List<ReactPackage> getPackages() {
      return Arrays.<ReactPackage>asList(
          new MainReactPackage(),
            new ReactNativeOneSignalPackage(),
            new RNSoundPackage(),
            new ReactNativeAudioPackage(),
            new RNFetchBlobPackage(),
            new RNFSPackage(),
            new PickerPackage(),
            new RNVersionCheckPackage(),
            new MPAndroidChartPackage(),
            new RNSpinkitPackage(),
            new RNGeocoderPackage(),
            new SvgPackage(),
            new MapsPackage()
      );
    }

    @Override
    protected String getJSMainModuleName() {
      return "index";
    }
  };

  @Override
  public ReactNativeHost getReactNativeHost() {
    return mReactNativeHost;
  }

  @Override
  public void onCreate() {
    super.onCreate();
    SoLoader.init(this, /* native exopackage */ false);
  }
}

AndroidManifest.xml

<manifest xmlns:android="http://schemas.android.com/apk/res/android"
  package="com.binomysteryshopper">

  <uses-permission android:name="android.permission.INTERNET" />
  <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />
  <uses-permission android:name="android.permission.SYSTEM_ALERT_WINDOW"/>
  <uses-permission android:name="android.permission.RECORD_AUDIO" />
  <uses-permission android:name="android.permission.RECORD_VIDEO"/>
  <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />
  <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION"/>
  <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION"/>
  <uses-permission android:name="android.permission.CAMERA"/>

  <uses-sdk
    android:minSdkVersion="16"
    android:targetSdkVersion="22" />

  <application
    android:name=".MainApplication"
    android:allowBackup="true"
    android:label="@string/app_name"
    android:icon="@mipmap/ic_launcher"
    android:largeHeap="true"
    android:theme="@style/AppTheme">

    <activity
      android:name=".MainActivity"
      android:launchMode="singleTop"
      android:label="@string/app_name"
      android:configChanges="keyboard|keyboardHidden|orientation|screenSize"
      android:screenOrientation="portrait"
      android:windowSoftInputMode="adjustResize"
      android:theme="@style/AppTheme">
        <intent-filter>
          <action android:name="android.intent.action.MAIN" />
          <action android:name="android.intent.action.DOWNLOAD_COMPLETE"/>
          <category android:name="android.intent.category.LAUNCHER" />
        </intent-filter>
        <intent-filter>
          <action android:name="android.intent.action.VIEW" />
          <category android:name="android.intent.category.DEFAULT" />
          <category android:name="android.intent.category.BROWSABLE" />
          <data android:scheme="binoshopper" />
        </intent-filter>
    </activity>
    <activity android:name="com.facebook.react.devsupport.DevSettingsActivity" />
  </application>
</manifest>

my OneSignal init

componentWillMount() {
    OneSignal.init(ONESIGNAL_APP_ID, {
      kOSSettingsKeyAutoPrompt: true,
      kOSSettingsKeyInAppLaunchURL: false,
      kOSSSettingsKeyPromptBeforeOpeningPushURL: true,
    })
    OneSignal.inFocusDisplaying(2)

    OneSignal.addEventListener('received', this.onPushNotificationReceived)
    OneSignal.addEventListener('opened', this.onPushNotificationOpened)
  }

componentWillUnmount() {
    OneSignal.removeEventListener('received', this.onPushNotificationReceived)
    OneSignal.removeEventListener('opened', this.onPushNotificationOpened)
  }

onPushNotificationReceived(notification) {
    Alert.alert('onReceived') // for tests on real device
  }

onPushNotificationOpened(openResult) {
    Alert.alert('onOpened') // for tests on real device
  }

[Ble3k]

On IOS it's working perfect. Unfortunately i can't provide logs from android logcat, because i don't have a real device. I was testing on my far away friend's device due TeamViewer.

I also tried to OneSignal.setLogLevel(0, 6) on a real device. But none errors were found, all requests succeeded, only events didn't fired.

[Nightsd01]

Hi @Ble3k , we have been quite busy working on a new Flutter SDK recently and haven't had time to investigate your issue yet, however I will investigate tomorrow and will get back to you shortly, apologies for the delay!

[slorber]

Hi,

Just wanted to say I see the same behavior on 3.2.5 on a dev device. Weird because I haven't changed anything recently and I'm pretty sure it worked some days ago, as my focused pushs are displayed as in-app toasts I remember having tested these and it was working.

I won't investigate further because we decided to move to normal push notifications even when app is focused

[Ble3k]

@Nightsd01 Thx, will wait for your investigation.

[sagarv1997]

On Android facing delay in receiving notifications, tested on my android phone and received notification after 30-40 mins, multiple times tested

[beisert1]

Any update on this? I am having the same issue on Android device (Samsung Galaxy S7).

None of the onOpened or onReceived events are being fired, however, the notification is being received. We need to be able to navigate the user to a specific screen when the onOpened event is fired.

[acamarjanovic]

Does anyone have problem with events (received, opened) not being fired after reload react native android app by double pressing R, or using react-native-restart package.

[slorber]

@ackosumane I actually use Expo and Updates.reload() very often in my dev workflow.

And you are right:

• I am able to get the callbacks fired on first app start
• I don't see the callbacks fired after the first app reload

@Nightsd01 So it looks like OneSignal RN Android integration currently does not support app reloads very well. It is not as impactful as I thought as most of the time the user will not reload the app, but there are production cases in which we want the user to reload the app (OverTheAir updates like Expo/CodePush, or user logout?). In such cases it would be great that after a reload the pushes are not broken ;)

[Ble3k]

@Nightsd01 Hi, any updates ?

[kashsbd]

Hello there, I also face the same problem.Any solutions for this ???

[Nightsd01]

As an update on this issue: we plan to fix this in the next update!

[sofianegargouri]

@Nightsd01 Any date ? Is there a branch we can fork with the fix ?

[Nightsd01]

@sofianegargouri Here's the PR: #592

We'll be releasing an update either tomorrow or the day after.