From 7e7fd51dd1f6a32422f8936d1983fb4a37bde7b8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Thu, 1 Jun 2023 15:49:19 +0200
Subject: [PATCH] Add comment about ShipmentServiceOptions being wrongly
 documented

---
 modules/stock_package_shipping_ups/stock.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/modules/stock_package_shipping_ups/stock.py b/modules/stock_package_shipping_ups/stock.py
index 5418dde100..2f1b613ff2 100644
--- a/modules/stock_package_shipping_ups/stock.py
+++ b/modules/stock_package_shipping_ups/stock.py
@@ -391,6 +391,8 @@ class CreateShippingUPS(Wizard):
             for p in packages]
         options = self.get_service_options(shipment)
         if options:
+            # options are set on package instead of shipment
+            # despite what UPS documentation says
             for pkg in packages:
                 pkg['ShipmentServiceOptions'] = options
         return {