Set options over explicit method.

Author: SebastianStehle

sample/Sample.Android/Resources/Resource.designer.cs

@@ -30,13 +30,19 @@ public partial interface INotifoMobilePush
     /// <summary>
     /// Method for pulling pending notifications.
     /// </summary>
-    /// <param name="options">The options for handling the pending notifications pull refresh request.</param>
     /// <returns>A <see cref="Task"/> representing the result of the asynchronous operation.</returns>
-    Task DidReceivePullRefreshRequestAsync(PullRefreshOptions? options = null);
+    Task DidReceivePullRefreshRequestAsync();
 
     /// <summary>
     /// Method for processing the user's response to a delivered notification.
     /// </summary>
     /// <param name="response">The user's response to the notification.</param>
     void DidReceiveNotificationResponse(UNNotificationResponse response);
+
+    /// <summary>
+    /// Sets the options that are used when new notifications are pulled.
+    /// </summary>
+    /// <param name="refreshOptions">The options.</param>
+    /// <returns>The current instance.</returns>
+    INotifoMobilePush SetRefreshOptions(PullRefreshOptions refreshOptions);
 }

sdk/Notifo.SDK/INotifoMobilePush.ios.cs

@@ -21,8 +21,16 @@ namespace Notifo.SDK.NotifoMobilePush;
 
 internal partial class NotifoMobilePushImplementation : NSObject
 {
+    private PullRefreshOptions refreshOptions;
     private INotificationHandler? notificationHandler;
 
+    /// <inheritdoc />
+    public INotifoMobilePush SetRefreshOptions(PullRefreshOptions refreshOptions)
+    {
+        this.refreshOptions = refreshOptions ?? new PullRefreshOptions();
+        return this;
+    }
+
     /// <inheritdoc />
     public INotifoMobilePush SetNotificationHandler(INotificationHandler? notificationHandler)
     {
@@ -53,29 +61,24 @@ public async Task DidReceiveNotificationRequestAsync(UNNotificationRequest reque
     }
 
     /// <inheritdoc />
-    public async Task DidReceivePullRefreshRequestAsync(PullRefreshOptions? options = null)
+    public async Task DidReceivePullRefreshRequestAsync()
     {
-        options ??= new PullRefreshOptions();
-
         // iOS does not maintain a queue of undelivered notifications, therefore we have to query here.
-        var notifications = await GetPendingNotificationsAsync(options.Take, options.Period, default);
+        var notifications = await GetPendingNotificationsAsync(refreshOptions.Take, refreshOptions.Period, default);
 
         foreach (var notification in notifications)
         {
-            if (options.RaiseEvent)
+            if (refreshOptions.RaiseEvent)
             {
                 OnReceived(new NotificationEventArgs(notification));
             }
 
-            if (notification.Silent)
+            if (notification.Silent || !refreshOptions.PresentNotification)
             {
                 continue;
             }
 
-            if (options.PresentNotification)
-            {
-                await ShowLocalNotificationAsync(notification);
-            }
+            await ShowLocalNotificationAsync(notification);
         }
 
         await TrackNotificationsAsync(notifications.ToArray());

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

@@ -12,7 +12,7 @@ namespace Notifo.SDK;
 /// <summary>
 /// Options for handling the pending notifications pull refresh request.
 /// </summary>
-public class PullRefreshOptions
+public sealed class PullRefreshOptions
 {
     /// <summary>
     /// The pull refresh request should raise the received event. Default: true.