setPriority option missing

[varunnayak]

setPriority(Notification.PRIORITY_MAX) can be used to show Heads-up Notifications

[jkasten2]

Thanks for the suggestion, we look into adding support for this in the future.

[nederev]

this is much needed. When do you think you will implement this?
Need this for cordova plugin as well

[jkasten2]

We are shooting to add this in the next few weeks.

[jkasten2]

I created a NotificationExtenderService that can be used to set this or to customize any other setting before our SDK displays the notification. Follow our Background Data and Notification Overriding to set this up. Step 3 shows a code example getting access to NotificationCompat.Builder where you can call setPriority.

We still plan to build setPriority in based on GCM priority but since there is now a supported way to customize features I am going to make the this lower priority.

[raysango]

@jkasten2 I'm trying to setPriority(2) to make the notification displays as a banner, I followed Background Data and Notification Overriding like this

public class NotificationExtenderPriority extends NotificationExtenderService {
    @Override
    protected boolean onNotificationProcessing(OSNotificationReceivedResult receivedResult) {
        OverrideSettings overrideSettings = new OverrideSettings();
        overrideSettings.extender = new NotificationCompat.Extender() {
            @Override
            public NotificationCompat.Builder extend(NotificationCompat.Builder builder) {
                // Sets the background notification color to Green on Android 5.0+ devices.
                builder.setPriority(2);
                return builder;
            }
        };
        return true;
    }
}

And then updated AndroidManifest.xml (I placed it in the MainActivity):

  <service android:exported="false" android:name="com.ray.app.NotificationExtenderPriority">
                <intent-filter>
                    <action android:name="com.onesignal.NotificationExtender" />
                </intent-filter>
            </service>

but I think something is missing, is there any other way to make the notification appears as a banner because currently I just get the icon on the status bar.

[jkasten2]

@raysango You need to add displayNotification(overrideSettings); before return true; to apply the overrideSettings.

[raysango]

@jkasten2 thanks, I added displayNotification(overrideSettings); but still the notification is displayed as icon in the status bar

[jkasten2]

@raysango Can you confirm that builder.setPriority(2); is running by adding Log.e(...) after this and checking for it in the logcat?

Also note Heads Up notifications were introduced in Android 5.0 Lollipop, the setting is ignored on older Android devices.

[raysango]

@jkasten2 It's not being called, I can't see the log after adding OSNotificationDisplayedResult displayedResult = displayNotification(overrideSettings); Log.d("OneSignalExample", "Notification displayed with id: " + displayedResult.androidNotificationId);

The test device I'm using is Android 6.01 Marshmallow

[jkasten2]

@raysango So it seems like your onNotificationProcessing method is being called but not the code in the extender? Note that this won't be called if you set an Android Group key from the dashboard or REST API for the notification. It is a current limitation of the SDK handling of grouped notifications.

[raysango]

@jkasten2 I'm using REST API to push notifications, but this is happening whether I assigned an android_group or not.

[jkasten2]

@raysango Can you confirm the below from my previous response?

So it seems like your onNotificationProcessing method is being called but not the code in the extender?`

Lastly can you make sure your using our latest 3.3.0 SDK? You can print OneSignal.VERSION to conform the version is "030300".