producteca 2.0.33__py3-none-any.whl → 2.0.57__py3-none-any.whl

producteca/abstract/abstract_dataclass.py

@@ -1,5 +1,6 @@
 from abc import ABC
 from ..config.config import ConfigProducteca
+from ..utils import clean_model_dump
 from dataclasses import dataclass
 from typing import Optional, Any
 
@@ -14,10 +15,11 @@ class BaseService(ABC):
         return repr(self._record)
 
     def to_dict(self):
-        return self._record.model_dump(by_alias=True)
+        return clean_model_dump(self._record)
 
     def to_json(self):
-        return self._record.model_dump_json(by_alias=True)
+        import json
+        return json.dumps(clean_model_dump(self._record))
 
     def __getattr__(self, key):
         return getattr(self._record, key)

producteca/payments/payments.py

@@ -20,14 +20,14 @@ class Payment(BaseModel):
     date: str
     amount: float
     couponAmount: Optional[float] = None
-    status: str
-    method: str
+    status: Optional[str] = None
+    method: Optional[str] = None
     integration: Optional[PaymentIntegration] = None
     transactionFee: Optional[float] = None
     installments: Optional[int] = None
     card: Optional[PaymentCard] = None
     notes: Optional[str] = None
-    hasCancelableStatus: bool
+    hasCancelableStatus: Optional[bool] = None
     id: Optional[int] = None
 
 

producteca/products/products.py

@@ -3,6 +3,7 @@ from pydantic import BaseModel, Field, ValidationError
 from dataclasses import dataclass
 from producteca.abstract.abstract_dataclass import BaseService
 from producteca.products.search_products import SearchProduct, SearchProductParams
+from producteca.utils import clean_model_dump
 import logging
 import requests
 
@@ -69,13 +70,13 @@ class Integration(BaseModel):
 class Variation(BaseModel):
     variation_id: Optional[int] = Field(default=None, alias='variationId')
     components: Optional[List] = None
-    pictures: Optional[List[Picture]] = None
-    stocks: Optional[List[Stock]] = None
+    pictures: Optional[Union[List[Picture], List]] = None
+    stocks: Optional[Union[List[Stock], List]] = None
     attributes_hash: Optional[str] = Field(default=None, alias='attributesHash')
     primary_color: Optional[str] = Field(default=None, alias='primaryColor')
     thumbnail: Optional[str] = None
-    attributes: Optional[List[Attribute]] = None
-    integrations: Optional[List[Integration]] = None
+    attributes: Optional[Union[List[Attribute], List]] = None
+    integrations: Optional[Union[List[Integration], List]] = None
     id: Optional[int] = None
     sku: Optional[str] = None
     barcode: Optional[str] = None
@@ -96,34 +97,35 @@ class BundleComponent(BaseModel):
 
 class BundleVariation(BaseModel):
     variation_id: int = Field(alias='variationId')
-    components: List[BundleComponent]
+    components: Union[List[BundleComponent], List]
 
 
 class BundleResult(BaseModel):
     company_id: int = Field(alias='companyId')
     product_id: int = Field(alias='productId')
-    variations: List[BundleVariation]
+    variations: Union[List[BundleVariation], List]
     id: str
 
 
 class BundleResponse(BaseModel):
-    results: List[BundleResult]
+    results: Union[List[BundleResult], List]
     count: int
 
 
 class Product(BaseModel):
-    integrations: Optional[List[Integration]] = None
-    variations: Optional[List[Variation]] = None
+    updatable_properties: Optional[List[str]] = Field(default=None, alias='$updatableProperties')
+    integrations: Optional[Union[List[Integration], List]] = None
+    variations: Optional[Union[List[Variation], List]] = None
     is_simple: Optional[bool] = Field(default=None, alias='isSimple')
     has_variations: Optional[bool] = Field(default=None, alias='hasVariations') 
     thumbnail: Optional[str] = None
     category: Optional[str] = None
     notes: Optional[str] = None
-    prices: Optional[List[Price]] = None
+    prices: Optional[Union[List[Price], List]] = None
     buying_price: Optional[float] = Field(default=None, alias='buyingPrice')
     is_archived: Optional[bool] = Field(default=None, alias='isArchived')
-    dimensions: Optional[Dimensions] = None
-    attributes: Optional[List[Attribute]] = None
+    dimensions: Optional[Union[Dimensions, dict]] = None
+    attributes: Optional[Union[List[Attribute], List]] = None
     metadata: Optional[List[str]] = None
     is_original: Optional[bool] = Field(default=None, alias='isOriginal')
     name: str
@@ -138,21 +140,21 @@ class ProductVariationBase(BaseModel):
     variation_id: Optional[int] = Field(default=None, alias='variationId')
     code: Optional[str] = None
     barcode: Optional[str] = None
-    attributes: List[Attribute] = []
+    attributes: Union[List[Attribute], List] = []
     tags: Optional[List[str]] = []
-    buying_price: Optional[float] = Field(0, alias='buyingPrice')
-    dimensions: Optional[Dimensions] = Field(default_factory=Dimensions)
+    buying_price: Optional[float] = Field(default=None, alias='buyingPrice')
+    dimensions: Optional[Union[Dimensions, dict]] = Field(default_factory=Dimensions)
     brand: Optional[str] = ''
     notes: Optional[str] = ''
-    deals: Optional[List[Deal]] = []
-    stocks: List[Stock]
-    prices: Optional[List[Price]] = []
-    pictures: Optional[List[Picture]] = []
+    deals: Optional[Union[List[Deal], List]] = []
+    stocks: Optional[Union[List[Stock], List]] = []
+    prices: Optional[Union[List[Price], List]] = []
+    pictures: Optional[Union[List[Picture], List]] = []
 
 
 class ProductVariation(ProductVariationBase):
     category: Optional[str] = Field(default=None)
-    name: str
+    name: Optional[str] = None
 
 
 class Shipping(BaseModel):
@@ -180,11 +182,11 @@ class MeliProduct(BaseModel):
     product_id: Optional[int] = Field(default=None, alias='productId')
     tags: Optional[List[str]] = Field(default=None)
     has_custom_shipping_costs: Optional[bool] = Field(default=None, alias='hasCustomShippingCosts')
-    shipping: Optional[Shipping] = None
-    mshops_shipping: Optional[MShopsShipping] = Field(default=None, alias='mShopsShipping')
+    shipping: Optional[Union[Shipping, dict]] = None
+    mshops_shipping: Optional[Union[MShopsShipping, dict]] = Field(default=None, alias='mShopsShipping')
     add_free_shipping_cost_to_price: Optional[bool] = Field(default=None, alias='addFreeShippingCostToPrice')
-    category: MeliCategory
-    attribute_completion: Optional[AttributeCompletion] = Field(default=None, alias='attributeCompletion')
+    category: Union[MeliCategory, dict]
+    attribute_completion: Optional[Union[AttributeCompletion, dict]] = Field(default=None, alias='attributeCompletion')
     catalog_products: Optional[List[str]] = Field(default=None, alias='catalogProducts')
     warranty: Optional[str] = None
     domain: Optional[str] = None
@@ -211,8 +213,8 @@ class ResolvedValue(BaseModel):
 
 class ResolvedError(BaseModel):
     resolved: Optional[bool] = None
-    reason: Optional[ErrorReason] = None
-    value: Optional[ResolvedValue] = None
+    reason: Optional[Union[ErrorReason, dict]] = None
+    value: Optional[Union[ResolvedValue, dict]] = None
     statusCode: Optional[int] = None
 
 
@@ -227,20 +229,20 @@ class ErrorContext(BaseModel):
 
 
 class SynchronizeResponse(BaseModel):
-    product: Optional[ResolvedError] = None
-    variation: Optional[ResolvedError] = None
-    deals: Optional[ResolvedError] = None
-    bundles: Optional[ResolvedError] = None
-    taxes: Optional[ResolvedError] = None
-    meliProductListingIntegrations: Optional[ResolvedError] = None
-    tags: Optional[ResolvedError] = None
-    productIntegrations: Optional[ResolvedError] = None
+    product: Optional[Union[ResolvedError, dict]] = None
+    variation: Optional[Union[ResolvedError, dict]] = None
+    deals: Optional[Union[ResolvedError, dict]] = None
+    bundles: Optional[Union[ResolvedError, dict]] = None
+    taxes: Optional[Union[ResolvedError, dict]] = None
+    meliProductListingIntegrations: Optional[Union[ResolvedError, dict]] = None
+    tags: Optional[Union[ResolvedError, dict]] = None
+    productIntegrations: Optional[Union[ResolvedError, dict]] = None
     statusCode: Optional[int] = None
-    error_context: Optional[ErrorContext] = Field(None, alias='error@context')
+    error_context: Optional[Union[ErrorContext, dict]] = Field(None, alias='error@context')
 
 
 class ListedSynchronizeResponse(BaseModel):
-    results: List[SynchronizeResponse]
+    results: Union[List[SynchronizeResponse], List]
 
 
 @dataclass
@@ -253,49 +255,51 @@ class ProductService(BaseService):
         return self
 
     def synchronize(self, payload) -> Union[Product, SynchronizeResponse]:
-
         endpoint_url = self.config.get_endpoint(f'{self.endpoint}/synchronize')
         headers = self.config.headers.copy()
         headers.update({"createifitdoesntexist": str(self.create_if_it_doesnt_exist).lower()})
         product_variation = ProductVariation(**payload)
         if not product_variation.code and not product_variation.sku:
             raise Exception("Sku or code should be provided to update the product")
-        data = product_variation.model_dump(by_alias=True, exclude_none=True)
+        # Hacer model_dump con limpieza automática de valores vacíos
+        data = clean_model_dump(product_variation)
+        _logger.info(f"Synchronizing product: {data}")
+        _logger.info(f"POST {endpoint_url} - Headers: {headers} - Data: {data}")
         response = requests.post(endpoint_url, json=data, headers=headers)
+        if not response.ok:
+            raise Exception(f"Error getting product {product_variation.sku} - {product_variation.code}\n {response.text}")
+        if response.status_code == 204:
+            _logger.info("Status code is 204 (No Content), product synchronized successfully but no changes were made")
+            return None
+        
+        _logger.info(f"response text: {response.text}")
         response_data = response.json()
-        try:
-            return Product(**response_data)
-        except ValidationError:
-            pass
+        _logger.debug(f"Response data: {response_data}")
         if isinstance(response_data, list):
             res = ListedSynchronizeResponse(results=response_data)
-            if any([r.error_context for r in res.results]):
+            if res.results and hasattr(res.results[0], 'error_context') and res.results[0].error_context:
                 raise Exception(f"Errored while updating {res.results[0].error_context} {res.model_dump_json()}")
-            else:
-                return res.results[0]
-        else:
-            try:
-                sync_resp = SynchronizeResponse(**response_data)
-                if sync_resp.error_context:
-                    raise Exception(f"Errored while updating {sync_resp.error_context} - {sync_resp.model_dump_json()}")
-                else:
-                    return sync_resp
-            except ValidationError:
-                try:
-                    error_res = ErrorReason(**response_data)
-                    raise Exception(f"Errored with the following message {error_res.message} - {error_res.model_dump_json()}")
-                except ValidationError:
-                    pass
+            return res.results[0] if res.results else None
 
-        if not response.ok:
-            raise Exception(f"Error getting product {product_variation.sku} - {product_variation.code}\n {response.text}")
-        if response.status_code == 204:
-            raise Exception("Status code is 204, meaning nothing was updated or created")
-        raise Exception(f"Unhandled error, check response {response.text}")
+        if isinstance(response_data, dict) and 'name' in response_data:
+            return Product(**response_data)
+
+        if isinstance(response_data, dict) and any(key in response_data for key in ['product', 'variation', 'statusCode']):
+            sync_resp = SynchronizeResponse(**response_data)
+            if sync_resp.error_context:
+                raise Exception(f"Errored while updating {sync_resp.error_context} - {sync_resp.model_dump_json()}")
+            return sync_resp
+
+        if isinstance(response_data, dict) and 'message' in response_data:
+            error_res = ErrorReason(**response_data)
+            raise Exception(f"Errored with the following message {error_res.message} - {error_res.model_dump_json()}")
+        
+        raise Exception(f"Unhandled response format, check response {response.text}")
 
     def get(self, product_id: int) -> "ProductService":
         endpoint_url = self.config.get_endpoint(f'{self.endpoint}/{product_id}')
         headers = self.config.headers
+        _logger.info(f"GET {endpoint_url} - Headers: {headers}")
         response = requests.get(endpoint_url, headers=headers)
         if not response.ok:
             raise Exception(f"Error getting product {product_id}\n {response.text}")
@@ -305,6 +309,7 @@ class ProductService(BaseService):
     def get_bundle(self, product_id: int) -> BundleResponse:
         endpoint_url = self.config.get_endpoint(f'{self.endpoint}/{product_id}/bundles')
         headers = self.config.headers
+        _logger.info(f"GET {endpoint_url} - Headers: {headers}")
         response = requests.get(endpoint_url, headers=headers)
         if not response.ok:
             raise Exception(f"Error getting bundle {product_id}\n {response.text}")
@@ -313,6 +318,7 @@ class ProductService(BaseService):
     def get_ml_integration(self, product_id: int) -> MeliProduct:
         endpoint_url = self.config.get_endpoint(f'{self.endpoint}/{product_id}/listingintegration')
         headers = self.config.headers
+        _logger.info(f"GET {endpoint_url} - Headers: {headers}")
         response = requests.get(endpoint_url, headers=headers)
         if not response.ok:
             raise Exception(f"Error getting ml integration {product_id}\n {response.text}")
@@ -323,7 +329,10 @@ class ProductService(BaseService):
         endpoint: str = f'search/{self.endpoint}'
         headers = self.config.headers
         url = self.config.get_endpoint(endpoint)
-        response = requests.get(url, headers=headers, params=params.model_dump(by_alias=True, exclude_none=True))
+        params_dict = clean_model_dump(params)
+        _logger.info(f"GET {url} - Headers: {headers} - Params: {params_dict}")
+        response = requests.get(url, headers=headers, params=params_dict)
+        _logger.info(f"Response status: {response.status_code} - Response text: {response.text}")
         if not response.ok:
             raise Exception(f"error in searching products {response.text}")
         return SearchProduct(**response.json())

producteca/products/search_products.py

@@ -56,24 +56,24 @@ class SearchDeals(BaseModel):
 class SearchResultItem(BaseModel):
     search_score: float = Field(..., alias='@search.score')
     id: int
-    product_id: int = Field(..., alias='productId')
-    company_id: int = Field(..., alias='companyId')
-    name: str
-    code: str
+    product_id: Optional[int] = Field(None, alias='productId')
+    company_id: Optional[int] = Field(None, alias='companyId')
+    name:Optional[str] = None
+    code: Optional[str] = None
     skus: List[str]
-    brand: str
-    category: str
-    thumbnail: str
-    stocks: List[SearchStocks]
-    warehouses_with_stock: List[str] = Field(..., alias='warehousesWithStock')
-    total_stock: int = Field(..., alias='totalStock')
-    has_pictures: bool = Field(..., alias='hasPictures')
-    buying_price: float = Field(..., alias='buyingPrice')
-    prices: List[SearchPrices]
-    integration_ids: List[str] = Field(..., alias='integrationIds')
-    integration_apps: List[str] = Field(..., alias='integrationApps')
-    integrations: List[SearchIntegration]
-    campaigns: List[str]
+    brand: Optional[str] = None
+    category: Optional[str] = None
+    thumbnail: Optional[str] = None
+    stocks: Optional[Union[List[SearchStocks], List]] = None
+    warehouses_with_stock: Optional[List[str]] = Field(None, alias='warehousesWithStock')
+    total_stock: Optional[int] = Field(None, alias='totalStock')
+    has_pictures: Optional[bool] = Field(None, alias='hasPictures')
+    buying_price: Optional[float] = Field(None, alias='buyingPrice')
+    prices: Optional[Union[List[SearchPrices], List]] = None
+    integration_ids: Optional[List[str]] = Field(None, alias='integrationIds')
+    integration_apps: Optional[List[str]] = Field(None, alias='integrationApps')
+    integrations: Optional[Union[List[SearchIntegration], List]] = None
+    campaigns: Optional[List[str]] = None
     app: Optional[int] = None
     status: Optional[str] = None
     synchronize_stock: Optional[bool] = Field(None, alias='synchronizeStock')
@@ -96,7 +96,7 @@ class SearchResultItem(BaseModel):
     mandatory_free_shipping: Optional[bool] = Field(None, alias='mandatoryFreeShipping')
     free_shipping: Optional[bool] = Field(None, alias='freeShipping')
     free_shipping_cost: Optional[float] = Field(None, alias='freeShippingCost')
-    template: Optional[str] = None
+    template: Optional[int] = None
     youtube_id: Optional[str] = Field(None, alias='youtubeId')
     warranty: Optional[str] = None
     permalink: Optional[str] = None
@@ -104,7 +104,7 @@ class SearchResultItem(BaseModel):
     attribute_completion_status: Optional[str] = Field(None, alias='attributeCompletionStatus')
     attribute_completion_count: Optional[int] = Field(None, alias='attributeCompletionCount')
     attribute_completion_total: Optional[int] = Field(None, alias='attributeCompletionTotal')
-    deals: Optional[SearchDeals] = None
+    deals: Optional[Union[SearchDeals, List]] = None
     campaign_status: Optional[List[str]] = Field(None, alias='campaignStatus')
     size_chart: Optional[str] = Field(None, alias='sizeChart')
     channel_status: Optional[List[str]] = Field(None, alias='channelStatus')

producteca/products/tests/test_products.py

@@ -53,16 +53,6 @@ class TestProduct(unittest.TestCase):
         
         self.assertEqual(response.sku, "TEST001")
 
-    @patch('requests.post')
-    def test_create_product_not_exist(self, mock_post):
-        # Mock product not found response
-        mock_response = Mock()
-        mock_response.status_code = 204
-        mock_post.return_value = mock_response
-
-        with self.assertRaises(Exception):
-            self.client.Product.synchronize(self.product_to_create_payload)
-
     @patch('requests.post')
     def test_update_product_success(self, mock_post):
         payload = self.product_to_create_payload

producteca/sales_orders/sales_orders.py

@@ -5,6 +5,7 @@ from producteca.abstract.abstract_dataclass import BaseService
 from producteca.sales_orders.search_sale_orders import SearchSalesOrderParams, SearchSalesOrder
 from producteca.payments.payments import Payment
 from producteca.shipments.shipment import Shipment
+from producteca.utils import clean_model_dump
 from dataclasses import dataclass
 import logging
 _logger = logging.getLogger(__name__)
@@ -193,10 +194,10 @@ class SaleOrderShipment(BaseModel):
 
 
 class SaleOrderInvoiceIntegrationAbstract(BaseModel):
-    id: int
-    integration_id: str = Field(alias="integrationId")
-    app: int
-    created_at: str = Field(alias="createdAt")
+    id: Optional[int] = None
+    integration_id: Optional[str] = Field(None, alias="integrationId")
+    app: Optional[int] = None
+    created_at: Optional[str] = Field(None, alias="createdAt")
     decrease_stock: Optional[bool] = Field(None, alias="decreaseStock")
 
 
@@ -287,9 +288,10 @@ class SaleOrderService(BaseService):
     def get(self, sale_order_id: int) -> "SaleOrderService":
         endpoint = f'{self.endpoint}/{sale_order_id}'
         url = self.config.get_endpoint(endpoint)
+        _logger.info(f"GET {url} - Headers: {self.config.headers}")
         response = requests.get(url, headers=self.config.headers)
         if not response.ok:
-            raise Exception(f"Order {sale_order_id} could not be fetched")
+            raise Exception(f"Order {sale_order_id} could not be fetched. Error {response.status_code} {response.text}")
         response_data = response.json()
         return self(**response_data)
 
@@ -298,6 +300,7 @@ class SaleOrderService(BaseService):
             raise Exception("You need to add a record id")
         endpoint = f'{self.endpoint}/{self._record.id}/labels'
         url = self.config.get_endpoint(endpoint)
+        _logger.info(f"GET {url} - Headers: {self.config.headers}")
         response = requests.get(url, headers=self.config.headers)
         if not response.ok:
             raise Exception("labels could not be gotten")
@@ -308,6 +311,7 @@ class SaleOrderService(BaseService):
             raise Exception("You need to add a record id")
         endpoint = f'{self.endpoint}/{self._record.id}/close'
         url = self.config.get_endpoint(endpoint)
+        _logger.info(f"POST {url} - Headers: {self.config.headers}")
         response = requests.post(url, headers=self.config.headers)
         if not response.ok:
             raise Exception("Order could not be closed")
@@ -317,6 +321,7 @@ class SaleOrderService(BaseService):
             raise Exception("You need to add a record id")
         endpoint = f'{self.endpoint}/{self._record.id}/cancel'
         url = self.config.get_endpoint(endpoint)
+        _logger.info(f"POST {url} - Headers: {self.config.headers}")
         response = requests.post(url, headers=self.config.headers)
         if not response.ok:
             raise Exception("Order could not be cancelled")
@@ -327,8 +332,10 @@ class SaleOrderService(BaseService):
         endpoint = f'{self.endpoint}/synchronize'
         url = self.config.get_endpoint(endpoint)
         # TODO: Check what can we sync, and what can we not sync
-        sync_body = SaleOrderSynchronize(**self._record.model_dump(by_alias=True))
-        response = requests.post(url, json=sync_body.model_dump(by_alias=True), headers=self.config.headers)
+        sync_body = SaleOrderSynchronize(**clean_model_dump(self._record))
+        sync_data = clean_model_dump(sync_body)
+        _logger.info(f"POST {url} - Headers: {self.config.headers} - Data: {sync_data}")
+        response = requests.post(url, json=sync_data, headers=self.config.headers)
         if not response.ok:
             raise Exception(f"Synchronize error {response.status_code} {response.text}")
         sync_res = SaleOrderSyncResponse(**response.json()) # noqa
@@ -337,11 +344,23 @@ class SaleOrderService(BaseService):
     def invoice_integration(self):
         if not self._record:
             raise Exception("You need to add a record id")
-        endpoint = f'{self.endpoint}/{self._record.id}/invoiceIntegration'
-        url = self.config.get_endpoint(endpoint)
-        response = requests.put(url, headers=self.config.headers,
-                                json={"id": self._record.id,
-                                      "invoiceIntegration": SaleOrderInvoiceIntegrationPut(**self._record.invoice_integration.model_dump(by_alias=True)).model_dump()})
+        
+        invoice_integration_data = clean_model_dump(self._record.invoice_integration)
+        
+        if self._record.invoice_integration.id:
+            endpoint = f'{self.endpoint}/{self._record.id}/invoiceIntegration'
+            url = self.config.get_endpoint(endpoint)
+            _logger.info(f"PUT {url} - Headers: {self.config.headers} - Data: {invoice_integration_data}")
+            response = requests.put(url, headers=self.config.headers,
+                                    json=invoice_integration_data)
+        else:
+            endpoint = f'{self.endpoint}/synchronize'
+            url = self.config.get_endpoint(endpoint)
+            sync_data = {"id": self._record.id, "invoiceIntegration": invoice_integration_data}
+            _logger.info(f"POST {url} - Headers: {self.config.headers} - Data: {sync_data}")
+            response = requests.post(url, headers=self.config.headers,
+                                     json=sync_data)
+        
         if not response.ok:
             raise Exception(f"Error on resposne {response.text}")
         return response.ok
@@ -351,6 +370,7 @@ class SaleOrderService(BaseService):
         headers = self.config.headers
         url = self.config.get_endpoint(endpoint)
         new_url = f"{url}?$filter={params.filter}&top={params.top}&skip={params.skip}"
+        _logger.info(f"GET {new_url} - Headers: {headers}")
         response = requests.get(
             new_url,
             headers=headers,
@@ -365,7 +385,9 @@ class SaleOrderService(BaseService):
             raise Exception("You need to add a record id")
         payment = Payment(**payload)
         url = self.config.get_endpoint(f"{self.endpoint}/{self._record.id}/payments")
-        res = requests.post(url, json=payment.model_dump(by_alias=True, exclude_none=True), headers=self.config.headers)
+        payment_data = clean_model_dump(payment)
+        _logger.info(f"POST {url} - Headers: {self.config.headers} - Data: {payment_data}")
+        res = requests.post(url, json=payment_data, headers=self.config.headers)
         if not res.ok:
             raise Exception(f"Error on resposne {res.text}")
         return Payment(**res.json())
@@ -375,7 +397,9 @@ class SaleOrderService(BaseService):
             raise Exception("You need to add a record id")
         payment = Payment(**payload)
         url = self.config.get_endpoint(f"{self.endpoint}/{self._record.id}/payments/{payment_id}")
-        res = requests.put(url, json=payment.model_dump(by_alias=True, exclude_none=True), headers=self.config.headers)
+        payment_data = clean_model_dump(payment)
+        _logger.info(f"PUT {url} - Headers: {self.config.headers} - Data: {payment_data}")
+        res = requests.put(url, json=payment_data, headers=self.config.headers)
         if not res.ok:
             raise Exception(f"Error on payment update {res.text}")
         return Payment(**res.json())
@@ -385,7 +409,9 @@ class SaleOrderService(BaseService):
             raise Exception("You need to add a record id")
         shipment = Shipment(**payload)
         url = self.config.get_endpoint(f"{self.endpoint}/{self._record.id}/shipments")
-        res = requests.post(url, json=shipment.model_dump(by_alias=True, exclude_none=True), headers=self.config.headers)
+        shipment_data = clean_model_dump(shipment)
+        _logger.info(f"POST {url} - Headers: {self.config.headers} - Data: {shipment_data}")
+        res = requests.post(url, json=shipment_data, headers=self.config.headers)
         if not res.ok:
             raise Exception(f"Error on shipment add {res.text}")
         return Shipment(**res.json())
@@ -395,7 +421,9 @@ class SaleOrderService(BaseService):
             raise Exception("You need to add a record id")
         shipment = Shipment(**payload)
         url = self.config.get_endpoint(f"{self.endpoint}/{self._record.id}/shipments/{shipment_id}")
-        res = requests.put(url, json=shipment.model_dump(by_alias=True, exclude_none=True), headers=self.config.headers)
+        shipment_data = clean_model_dump(shipment)
+        _logger.info(f"PUT {url} - Headers: {self.config.headers} - Data: {shipment_data}")
+        res = requests.put(url, json=shipment_data, headers=self.config.headers)
         if not res.ok:
             raise Exception(f"Error on shipment update {res.text}")
         return Shipment(**res.json())

producteca/sales_orders/search_sale_orders.py

@@ -8,7 +8,7 @@ _logger = logging.getLogger(__name__)
 class SalesOrderProduct(BaseModel):
     id: int
     name: str
-    code: str
+    code: Optional[str] = None
     brand: Optional[str] = None
 
 
@@ -20,8 +20,8 @@ class SalesOrderVariationAttribute(BaseModel):
 class SalesOrderVariation(BaseModel):
     id: int
     attributes: Optional[List[SalesOrderVariationAttribute]] = None
-    sku: str
-    thumbnail: str
+    sku: Optional[str] = None
+    thumbnail: Optional[str] = None
 
 
 class SalesOrderLine(BaseModel):
@@ -48,15 +48,15 @@ class SalesOrderPaymentIntegration(BaseModel):
 class SalesOrderPayment(BaseModel):
     date: str
     amount: float
-    coupon_amount: float = Field(alias="couponAmount")
-    status: str
+    coupon_amount: Optional[float] = Field(default=None, alias="couponAmount")
+    status: Optional[str] = None
     method: str
     integration: Optional[SalesOrderPaymentIntegration] = None
-    transaction_fee: float = Field(alias="transactionFee")
-    installments: int
+    transaction_fee: Optional[float] = Field(default=None, alias="transactionFee")
+    installments: Optional[int] = None
     card: Optional[SalesOrderCard] = None
     notes: Optional[str] = None
-    has_cancelable_status: bool = Field(alias="hasCancelableStatus")
+    has_cancelable_status: Optional[bool] = Field(default=None, alias="hasCancelableStatus")
     id: int
 
 

producteca/utils.py

@@ -0,0 +1,50 @@
+"""
+Utility functions for the producteca package
+"""
+from typing import Any, Dict, List, Union
+from pydantic import BaseModel
+
+
+def exclude_empty_values(obj: Any) -> Any:
+    """Recursively remove None, empty lists, empty strings, and empty dicts
+    
+    Note: Preserves 0 values for numeric fields as they are valid prices/quantities
+    Special case: Preserves empty list [] for 'updatableProperties' to allow explicit updates
+    """
+    if isinstance(obj, dict):
+        filtered_dict = {}
+        for k, v in obj.items():
+            # Special case: preserve empty list for updatableProperties
+            # This allows explicitly sending [] to clear/update the field
+            if k == 'updatableProperties' and v == []:
+                filtered_dict[k] = v
+                continue
+            
+            # Skip None, empty lists, empty strings, empty dicts
+            if v is None or v == [] or v == "" or v == {}:
+                continue
+            # Keep numeric 0 values - they are valid for prices, quantities, etc.
+            filtered_dict[k] = exclude_empty_values(v)
+        return filtered_dict
+    elif isinstance(obj, list):
+        filtered_list = [exclude_empty_values(item) for item in obj if item is not None]
+        return [item for item in filtered_list if item != [] and item != "" and item != {}]
+    else:
+        return obj
+
+
+def clean_model_dump(model: BaseModel, by_alias: bool = True, exclude_none: bool = True, **kwargs) -> Dict[str, Any]:
+    """
+    Enhanced model_dump that automatically cleans empty values
+    
+    Args:
+        model: Pydantic model instance
+        by_alias: Use field aliases in output
+        exclude_none: Exclude None values
+        **kwargs: Additional arguments to pass to model_dump
+    
+    Returns:
+        Clean dictionary with empty values removed
+    """
+    raw_data = model.model_dump(by_alias=by_alias, exclude_none=exclude_none, **kwargs)
+    return exclude_empty_values(raw_data)

{producteca-2.0.33.dist-info → producteca-2.0.57.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: producteca
-Version: 2.0.33
+Version: 2.0.57
 Summary: 
 Author: Chroma Agency, Matias Rivera
 Author-email: mrivera@chroma.agency

{producteca-2.0.33.dist-info → producteca-2.0.57.dist-info}/RECORD

@@ -1,22 +1,22 @@
 producteca/__init__.py,sha256=ML-Y_hCd8HOgNOVGiqGpbLqWAtvPXqQQaBQmj-A5Lvc,52
 producteca/abstract/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-producteca/abstract/abstract_dataclass.py,sha256=LpAsIdVgCbqT602_1X19mfnuooa5j6LttjvZKKJhw-Q,549
+producteca/abstract/abstract_dataclass.py,sha256=Axsf4ewa7Qua5aosvEXksR-fXoGF01_EUlOl1oKdkIk,597
 producteca/client.py,sha256=4T4vLh_Zz56mG2S0P9Il1NEJ4Rj0g2mgXYA7KJjY72U,789
 producteca/config/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 producteca/config/config.py,sha256=uTRaHLI9L7P_LPuFmuYgV50Aedz9MfDbXOZVZPFkOuI,658
 producteca/payments/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-producteca/payments/payments.py,sha256=ac0X-QC4CGvxs40B_qWv0IepBVpj_nULNSNp8s3v5aA,842
+producteca/payments/payments.py,sha256=zhbv6zWb1i04wGdqiyQJ8Dqp435xNHEOsjnps-pAT58,893
 producteca/payments/tests/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 producteca/payments/tests/test_payments.py,sha256=wutk5zK2xRChTvKyTmi54Y_92PzNtUKZF6xWF298sFY,3032
 producteca/products/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-producteca/products/products.py,sha256=FQeif3q52JGUcYVu5VKzY81FPSInQlc-HfYpYFlWmxo,12403
-producteca/products/search_products.py,sha256=qoo7ttdBuDUKEIOz4ysuBwK88aJ6PfSOtLkgNymvD4Y,5880
+producteca/products/products.py,sha256=G5ApUxeipHKRCQZnGdCYLxNE-m--6dOsYiQB_sMI_Dw,13835
+producteca/products/search_products.py,sha256=wPrnChvzPKzVermtKbi-V7wBmILJhJToyoNtpOOeXc4,6172
 producteca/products/tests/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-producteca/products/tests/test_products.py,sha256=0Tph7T9D2XTLdTDoIpR2PcLTrvPt6h2LPuKowTXrDgU,5742
+producteca/products/tests/test_products.py,sha256=JMsWq9DDJZn8If1dz42WSJ93-akOm7flFQPm6WYeJus,5377
 producteca/products/tests/test_search_products.py,sha256=ISkDClYaYnwo23a7_O4OlgPfc4dAZCMinLAi07yecEI,4941
 producteca/sales_orders/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-producteca/sales_orders/sales_orders.py,sha256=AQKCujjNRArzzmMP89eHnwypRLVR67A9PBvIkazcebo,16472
-producteca/sales_orders/search_sale_orders.py,sha256=oVegsfQ1vVrydyBbgbvc30TgQaufCnyEJfm8hOWWFfs,5022
+producteca/sales_orders/sales_orders.py,sha256=js7OoCcSn6P-eHa9IwJDym7kYa5EUteTwI8563VVxEo,18027
+producteca/sales_orders/search_sale_orders.py,sha256=rvbPqVqYjRpM45QXUZp_6uCk1j7CRyfN5hKsinvj6Kc,5179
 producteca/sales_orders/tests/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 producteca/sales_orders/tests/search.json,sha256=v6LkTvQyeBdqxRc3iWwXH5Ugs7EYJAEOiuPuuj_rHYk,3508
 producteca/sales_orders/tests/test_sales_orders.py,sha256=hPtUqp4GUpp9iBr3AMGsS7nxHQ5JHJy-DLuTxf9Eufs,4245
@@ -25,8 +25,9 @@ producteca/shipments/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hS
 producteca/shipments/shipment.py,sha256=YokJRBX1ZbUtZVc4AOAB188rbHFO0_MH5hFv5U9Qpjw,844
 producteca/shipments/tests/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 producteca/shipments/tests/test_shipment.py,sha256=0edX3WM-t4nbp53r_Lvhtk_Suc2QCnIUQ27I90qSUfE,2512
-producteca-2.0.33.dist-info/LICENSE,sha256=DZak_2itbUtvHzD3E7GNUYSRK6jdOJ-GqncQ2weavLA,34523
-producteca-2.0.33.dist-info/METADATA,sha256=czmxx5JQJFyxSAYgUUN7RieVrHgtKcx9x6vqaztGsHQ,3809
-producteca-2.0.33.dist-info/WHEEL,sha256=FMvqSimYX_P7y0a7UY-_Mc83r5zkBZsCYPm7Lr0Bsq4,88
-producteca-2.0.33.dist-info/entry_points.txt,sha256=BFSDFLbB70p8YVZPiU7HDJdj2VyyLdMVa4ekLQAUAVc,125
-producteca-2.0.33.dist-info/RECORD,,
+producteca/utils.py,sha256=krCUtr5cwu1787v24wSagSltPjY1ERe3E1whhT-SZKU,1961
+producteca-2.0.57.dist-info/LICENSE,sha256=DZak_2itbUtvHzD3E7GNUYSRK6jdOJ-GqncQ2weavLA,34523
+producteca-2.0.57.dist-info/METADATA,sha256=hfixRYOulnUNZD58c7LEytb-rke71V8z1bSoQP6m5hQ,3809
+producteca-2.0.57.dist-info/WHEEL,sha256=FMvqSimYX_P7y0a7UY-_Mc83r5zkBZsCYPm7Lr0Bsq4,88
+producteca-2.0.57.dist-info/entry_points.txt,sha256=BFSDFLbB70p8YVZPiU7HDJdj2VyyLdMVa4ekLQAUAVc,125
+producteca-2.0.57.dist-info/RECORD,,