- Remove Shipment.CreateReturn function

- Remove now-unneeded nullability utilities
- Update CHANGELOG.md with info about nullable properties

CHANGELOG.md

@@ -12,6 +12,9 @@
   - Some properties have been renamed to avoid naming conflicts:
     - `Rate.rate` is now `Rate.Price`
     - `Message.message` is now `Message.Text`
+- All properties are now nullable.
+  - Almost all properties will be assigned a value during JSON deserialization. This is mostly to address compiler warnings.
+  - Users can proceed with the assumption that any given property will not be null.
 - All objects now share a common base set of properties, including `Id`, `CreatedAt`, `UpdatedAt`, and `Mode`.
 - Under the hood improvements:
   - Underlying `Request`-`Client`-`ClientConfiguration` relationship has been re-architected to allow for thread safety.

EasyPost/Models/API/Shipment.cs

@@ -75,22 +75,6 @@ public class Shipment : EasyPostObject
 
         #region CRUD Operations
 
-        /// <summary>
-        ///     Return this shipment.
-        /// </summary>
-        /// <returns>A return shipment.</returns>
-        [CrudOperations.Create]
-        public async Task<Shipment> Return()
-        {
-            if (Id == null)
-            {
-                // This is a local object, not one pulled from the server.
-                throw new Exception("id is null");
-            }
-
-            return await (Client as Client)!.Shipment.CreateReturn(this);
-        }
-
         /// <summary>
         ///     Get the Smartrates for this shipment.
         /// </summary>

EasyPost/Services/ShipmentService.cs

@@ -47,41 +47,6 @@ public async Task<Shipment> Create(Dictionary<string, object> parameters, bool w
             return await Create<Shipment>("shipments", parameters);
         }
 
-        /// <summary>
-        ///     Create a return shipment for a shipment.
-        ///     Uses the same To and From addresses as the original shipment by default.
-        ///     Users can override the To and From addresses by passing in new addresses.
-        /// </summary>
-        /// <param name="shipment">Shipment to return.</param>
-        /// <param name="overrideToAddress">Override the To address for the return.</param>
-        /// <param name="overrideFromAddress">Override the From address for the return.</param>
-        /// <returns>A return shipment.</returns>
-        [CrudOperations.Create]
-        public async Task<Shipment> CreateReturn(Shipment shipment, Address? overrideToAddress = null, Address? overrideFromAddress = null)
-        {
-            if (shipment.Id == null)
-            {
-                throw new Exception("Shipment.id is null.");
-            }
-
-            overrideToAddress ??= shipment.FromAddress; // Use override address if provided, otherwise use original shipment From address as return To address.
-            overrideFromAddress ??= shipment.ToAddress; // Use override address if provided, otherwise use original shipment To address as return From address.
-
-            if (Nullability.AnyAreNull(overrideToAddress, overrideFromAddress, shipment.Parcel))
-            {
-                throw new Exception("Shipment missing required elements.");
-            }
-
-            return await Create(
-                new Dictionary<string, object>
-                {
-                    { "to_address", overrideToAddress! },
-                    { "from_address", overrideFromAddress! },
-                    { "parcel", shipment.Parcel! },
-                    { "is_return", true },
-                });
-        }
-
         /// <summary>
         ///     Get a paginated list of shipments.
         /// </summary>