From dee5c880c9bf309cb67f6cbd04747aa76c259dbd Mon Sep 17 00:00:00 2001
From: Chris <reichelt@easierlife.de>
Date: Thu, 26 Jan 2023 10:19:50 +0100
Subject: [PATCH] add comment

---
 Directory.Build.props                                           | 2 +-
 .../NotifoMobilePush/NotifoMobilePushImplementation.ios.cs      | 2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/Directory.Build.props b/Directory.Build.props
index ae3d33e..783faff 100644
--- a/Directory.Build.props
+++ b/Directory.Build.props
@@ -8,7 +8,7 @@
     <PackageLicenseExpression>MIT</PackageLicenseExpression>
     <PackageProjectUrl>https://notifo.io</PackageProjectUrl>
     <PackageTags>notifo xamarin firebase</PackageTags>
-    <Version>1.2.1-beta1</Version>
+    <Version>1.2.1-beta2</Version>
     <LangVersion>9</LangVersion>
   </PropertyGroup>
 </Project>
diff --git a/sdk/Notifo.SDK.Core/NotifoMobilePush/NotifoMobilePushImplementation.ios.cs b/sdk/Notifo.SDK.Core/NotifoMobilePush/NotifoMobilePushImplementation.ios.cs
index d3d96d3..c794f28 100644
--- a/sdk/Notifo.SDK.Core/NotifoMobilePush/NotifoMobilePushImplementation.ios.cs
+++ b/sdk/Notifo.SDK.Core/NotifoMobilePush/NotifoMobilePushImplementation.ios.cs
@@ -155,6 +155,8 @@ private async Task<List<UserNotificationDto>> GetPendingNotifications1_0Async(in
         private async Task<List<UserNotificationDto>> GetPendingNotifications1_4Async(int take, DateTime after,
             CancellationToken ct)
         {
+            // use DeviceIdentifier because Token could change over time and you could miss pending notifications. DeviceIdentifier should be more stable.
+            // take is not returning the exact amount of entries. It's taking the X last entries, then filters for messages that are unseen.
             var result = await Client.Notifications.GetMyDeviceNotificationsAsync(DeviceIdentifier, after, true, take * 2, ct);
 
             return result.Items;