[Enhancement] Create Channels Group

[eyeveye]

Summary

For Android to be able to group the channels into group.

[thomasgalliker]

Can you provide some more info, so I dont have to do the research work. I have never used channel groups before.

[eyeveye]

Hi,

Currently my code as follow:

I am using Plugin.Firebase.CloudMessaging. As you can see, there is a group called "Latest Result" which is the group for the rest of the channel.

private void CreateNotificationChannel()
{

    notificationManager.CreateNotificationChannelGroup(new NotificationChannelGroup(NotificationConstants.NOTIFICATION_LATEST_RESULT_GROUP_CHANNEL_ID, "Latest Result"));

    notificationManager.CreateNotificationChannel(new NotificationChannel(
    NotificationConstants.NOTIFICATION_LATEST_4D_CHANNEL_ID,
    "4D",
    NotificationImportance.High)
    { Group = NotificationConstants.NOTIFICATION_LATEST_RESULT_GROUP_CHANNEL_ID });

    notificationManager.CreateNotificationChannel(new NotificationChannel(
    NotificationConstants.NOTIFICATION_LATEST_TOTO_CHANNEL_ID,
    "TOTO",
    NotificationImportance.High)
    { Group = NotificationConstants.NOTIFICATION_LATEST_RESULT_GROUP_CHANNEL_ID });

    notificationManager.CreateNotificationChannel(new NotificationChannel(
    NotificationConstants.NOTIFICATION_LATEST_SWEEP_CHANNEL_ID,
    "SWEEP",
    NotificationImportance.High)
    { Group = NotificationConstants.NOTIFICATION_LATEST_RESULT_GROUP_CHANNEL_ID });

}

In Android App Notification Settings it will show as such:

[thomasgalliker]

I‘ll have a look at it.

[thomasgalliker]

I cannot find any hints regarding CreateNotificationChannelGroup in Plugin.Firebase.CloudMessaging. You probably use the Plugin.LocalNotification? Nevertheless, it's quite a straightforward implementation. I give it a try.

[thomasgalliker]

@eyeveye do you want to try the latest 3.x pre release? I extended the INotificationChannels interface with channel group operations. Also, you can set the initial list of groups and channels via options (see MauiProgram of the sample app).

Question: Since you're using notification channels in your app --> Would you expect that the Plugin.FirebasePushNotifications library creates a default notification channel by itself (automatically at startup time) or would you leave this to the dev of the app if a defaul notification channel needs to be created or not. (The current 3.1.x pre release no longer creates a default notification channel. You have to do it by yourself if you want one...). I'm not 100% sure what is the better option. Not everyone would use notification channels I guess...

[thomasgalliker]

The brach is ready to be merged. If you want to have a look at the latest 3.1.x pre-release, I'd highly appreciate it.

[thomasgalliker]

@eyeveye could you have a look at the latest 3.1.x pre-release?
Happy new year 🎉

[thomasgalliker]

@eyeveye can you have a look at the latest 3.1.x-pre release of the nuget and let me know if the changes match your needs? Also, if you want to, have a look at the code changes in #87. From my point of view it looks good to merge it.

[eyeveye]

I have tried to assigned the group in the
yield return new NotificationChannelRequest
{
ChannelId = NotificationConstants.NOTIFICATION_DEFAULT_CHANNEL_ID,
ChannelName = "General",
Description = "General Channel",
LockscreenVisibility = NotificationVisibility.Public,
Importance = NotificationImportance.High,
Group = NotificationConstants.NOTIFICATION_OTHER_GROUP_CHANNEL_ID
//IsDefault = true
};

I can't see it from the Notification Categories in the phone.

[thomasgalliker]

Did you also create the channel group? Do you have a small repro app? Or did you try the sample app in this feature branch?

[eyeveye]

Did you also create the channel group? Do you have a small repro app? Or did you try the sample app in this feature branch?

This is good. I have tested it.