Merge pull request #28 from Mark-RSK/22-Deallimitisincorrect

Added constructors to ListRequestOptions to set the upper limit for deals

Author: clarkd

HubSpot.NET.Examples/Deals.cs

@@ -33,7 +33,7 @@ public static void Example()
              * Get all deals
              */
             var deals = api.Deal.List<DealHubSpotModel>(false,
-                new ListRequestOptions { PropertiesToInclude = new List<string> { "dealname", "amount" } });
+                new ListRequestOptions(250) { PropertiesToInclude = new List<string> { "dealname", "amount" } });
 
             /**
              *  Get all deals

HubSpot.NET/Api/Deal/HubSpotDealApi.cs

@@ -73,7 +73,7 @@ public HubSpotDealApi(IHubSpotClient client)
         {
             if (opts == null)
             {
-                opts = new ListRequestOptions();
+                opts = new ListRequestOptions(250);
             }
 
             var path = $"{new DealListHubSpotModel<T>().RouteBasePath}/deal/paged"

HubSpot.NET/Core/ListRequestOptions.cs

@@ -9,6 +9,7 @@ namespace HubSpot.NET.Core
     public class ListRequestOptions
     {
         private int _limit = 20;
+        private readonly int _upperLimit;
 
         /// <summary>
         /// Gets or sets the number of items to return.
@@ -24,15 +25,34 @@ public int Limit
             get => _limit;
             set
             {
-                if (value < 1 || value > 100)
+                if (value < 1 || value > _upperLimit)
                 {
                     throw new ArgumentException(
-                        $"Number of items to return must be a positive ingeteger greater than 0, and less than 100 - you provided {value}");
+                        $"Number of items to return must be a positive integer greater than 0, and less than {_upperLimit} - you provided {value}");
                 }
                 _limit = value;
             }
         }
 
+        /// <summary>
+        /// Initializes a new instance of the <see cref="T:HubSpot.NET.Core.ListRequestOptions"/> class.
+        /// </summary>
+        /// <param name="upperLimit">Upper limit for the amount of items to request for the list.</param>
+        public ListRequestOptions(int upperLimit)
+        {
+            _upperLimit = upperLimit;
+        }
+
+        /// <summary>
+        /// Initializes a new instance of the <see cref="T:HubSpot.NET.Core.ListRequestOptions"/> class.
+        /// Sets the upper limit to 100 
+        /// </summary>
+        public ListRequestOptions()
+            : this(100)
+        {
+
+        }
+
         /// <summary>
         /// Get or set the continuation offset when calling list many times to enumerate all your items
         /// </summary>