Fix tracking.

Author: SebastianStehle

sdk/Notifo.SDK/Extensions/TaskExtensions.cs

@@ -13,6 +13,27 @@ namespace Notifo.SDK.Extensions;
 
 internal static class TaskExtensions
 {
+    private static readonly Action<Task> IgnoreTaskContinuation = t => { var ignored = t.Exception; };
+
+    public static void Forget(this Task task)
+    {
+        if (task.IsCompleted)
+        {
+#pragma warning disable IDE0059 // Unnecessary assignment of a value
+            var ignored = task.Exception;
+#pragma warning restore IDE0059 // Unnecessary assignment of a value
+        }
+        else
+        {
+            task.ContinueWith(
+                IgnoreTaskContinuation,
+                CancellationToken.None,
+                TaskContinuationOptions.OnlyOnFaulted |
+                TaskContinuationOptions.ExecuteSynchronously,
+                TaskScheduler.Default);
+        }
+    }
+
     public static async Task WithCancellation(this Task task, CancellationToken cancellationToken)
     {
         var tcs = new TaskCompletionSource<bool>();

sdk/Notifo.SDK/NotifoMobilePush/NotifoMobilePushImplementation.android.cs

@@ -43,10 +43,7 @@ public INotifoMobilePush SetImageCacheCapacity(int capacity)
 
     private void PushEventsProvider_OnNotificationReceivedAndroid(object sender, NotificationEventArgs e)
     {
-        if (!string.IsNullOrWhiteSpace(e.Notification.TrackSeenUrl))
-        {
-            _ = TrackNotificationsAsync(e.Notification);
-        }
+        TrackNotificationsAsync(e.Notification).Forget();
     }
 
     internal void OnBuildNotification(NotificationCompat.Builder notificationBuilder, UserNotificationDto notification)

sdk/Notifo.SDK/NotifoMobilePush/NotifoMobilePushImplementation.cs

@@ -11,6 +11,7 @@
 using System.Threading;
 using System.Threading.Tasks;
 using Notifo.SDK.CommandQueue;
+using Notifo.SDK.Extensions;
 using Notifo.SDK.Helpers;
 using Notifo.SDK.PushEventProvider;
 using Notifo.SDK.Resources;
@@ -196,7 +197,7 @@ async Task RegisterAsync()
             Register(tokenToRegister);
         }
 
-        _ = RegisterAsync();
+        RegisterAsync().Forget();
     }
 
     /// <inheritdoc/>

sdk/Notifo.SDK/NotifoMobilePush/NotifoMobilePushImplementation.ios.cs

@@ -60,25 +60,29 @@ public async Task DidReceivePullRefreshRequestAsync(PullRefreshOptions? options
         // iOS does not maintain a queue of undelivered notifications, therefore we have to query here.
         var notifications = await GetPendingNotificationsAsync(options.Take, options.Period, default);
 
+        List<UserNotificationDto>? trackImmediatly = null;
+
         foreach (var notification in notifications)
         {
             if (options.RaiseEvent)
             {
                 OnReceived(new NotificationEventArgs(notification));
             }
 
-            if (notification.Silent)
+            if (notification.Silent || !options.PresentNotification)
             {
+                trackImmediatly ??= new List<UserNotificationDto>();
+                trackImmediatly.Add(notification);
                 continue;
             }
 
-            if (options.PresentNotification)
-            {
-                await ShowLocalNotificationAsync(notification);
-            }
+            await ShowLocalNotificationAsync(notification);
         }
 
-        await TrackNotificationsAsync(notifications.ToArray());
+        if (trackImmediatly != null)
+        {
+            await TrackNotificationsAsync(notifications.ToArray());
+        }
     }
 
     private async Task<IEnumerable<UserNotificationDto>> GetPendingNotificationsAsync(int take, TimeSpan maxAge,
@@ -167,6 +171,10 @@ private async Task ShowLocalNotificationAsync(UserNotificationDto notification)
             {
                 NotifoIO.Current.RaiseError(error.LocalizedDescription, null, this);
             }
+            else
+            {
+                TrackNotificationsAsync(notification).Forget();
+            }
         });
     }
 
@@ -240,7 +248,7 @@ private async Task EnrichBasicAsync(UNMutableNotificationContent content, UserNo
             UNUserNotificationCenter.Current.SetNotificationCategories(new NSSet<UNNotificationCategory>(categories.ToArray()));
 
             // Without this call action buttons will not be added or updated.
-            _ = await UNUserNotificationCenter.Current.GetNotificationCategoriesAsync();
+            await UNUserNotificationCenter.Current.GetNotificationCategoriesAsync();
 
             content.CategoryIdentifier = categoryId;
         }