Not Required added in reservation

Author: MySecondLanguage

nxtbn/order/__init__.py

@@ -147,13 +147,14 @@ class OrderStockReservationStatus(models.TextChoices):
     OrderReservationStatus is an enumeration of possible statuses for an order reservation.
 
     Attributes:
-        RESERVED: The stock has been reserved for the order.
-        RELEASED: The stock reservation has been released.
-        INSUFFICIENT_STOCK: The stock is insufficient to fulfill the order.
-        NOT_RESERVED: The stock has not been reserved for the order.
+        RESERVED: Stock has been reserved for the order.
+        RELEASED: Stock reservation has been released.
+        FAILED: Stock reservation has failed due to insufficient stock.
+        NOT_REQUIRED: Stock reservation is not required.
+        DISPATCHED: Stock has been dispatched for the order.
     """
     RESERVED = 'RESERVED', _('Reserved')
     RELEASED = 'RELEASED', _('Released') # Re-adjust stock after order is cancelled
     FAILED = 'FAILED', _('Failed') # If failed, that is mean stock is insufficient to fulfill the order, have to fixed it before proceed
-    NOT_RESERVED = 'NOT_RESERVED', _('Not Reserved') # DO NOTHING IF NOT RESERVED, NO NEED VALIDATION
+    NOT_REQUIRED = 'NOT_REQUIRED', _('Not Required') # DO NOTHING IF NOT REQUIRED, NO NEED VALIDATION
     DISPATCHED = 'DISPATCHED', _('Dispatched')

nxtbn/order/migrations/0040_alter_order_reservation_status.py

@@ -0,0 +1,28 @@
+# Generated by Django 4.2.11 on 2025-02-21 09:22
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("order", "0039_alter_order_options"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="order",
+            name="reservation_status",
+            field=models.CharField(
+                choices=[
+                    ("RESERVED", "Reserved"),
+                    ("RELEASED", "Released"),
+                    ("FAILED", "Failed"),
+                    ("NOT_REQUIRED", "Not Required"),
+                    ("DISPATCHED", "Dispatched"),
+                ],
+                default="NOT_REQUIRED",
+                max_length=20,
+            ),
+        ),
+    ]

nxtbn/order/models.py

@@ -241,7 +241,7 @@ class Order(MonetaryMixin, AbstractBaseUUIDModel):
         help_text="Preferred payment method for this order. The actual payment method may differ when the order is initiated or paid."
     )
     reservation_status = models.CharField(
-        default=OrderStockReservationStatus.NOT_RESERVED,
+        default=OrderStockReservationStatus.NOT_REQUIRED,
         max_length=20,
         choices=OrderStockReservationStatus.choices
     )

nxtbn/warehouse/utils.py

@@ -106,7 +106,7 @@ def release_stock(order):
         return order
 
 def deduct_reservation_on_packed_for_dispatch(order):
-    if order.reservation_status == OrderStockReservationStatus.NOT_RESERVED: # As not reservable, nothing to do
+    if order.reservation_status == OrderStockReservationStatus.NOT_REQUIRED: # As not reservable, nothing to do
         return None
 
     if order.reservation_status == OrderStockReservationStatus.FAILED: