fourzeroone/python_dhl_germany

python_dhl_germany's Issues

create classes for shipper, receiver, order, and position

Wie wäre es wenn shipper, receiver und order seine eigene Klassen haben. 2 Vorteile sehe ich:

• Bei der Erstellung ihre Instances kann man in einer IDE wie PyCharm die Attributes sehen. Als ich deine Repo über DHL Forum gefunden habe, wusste ich nicht wie die shipper, receiver und order aussehen sollen, bis ich Beispiele in den Testen gefunden habe und irgendwann danach hast du ein Beispiel in README reingepackt.
• Indirekte Dokumentation, welche Felder + wie die Struktur zu erwarten sind.

ok wenn man dieses Design macht, dann braucht man auch die Klasse Position als input für Order

Was hälst du davon?
VG
Vinh

https://github.com/FourZeroOne/python_dhl_germany/blob/master/README.md#usage

make weight consistent

Hi Johannes,

erstmal danke für deine Repo. Es hat mir sehr viel geholfen. Ich würde gerne etwas dazu beitragen und würde dir meine Vorschläge erstmal vorstellen. Wenn es passt, implementiere ich und schicke dir einen Pull Request:

• "netWeightInKG": position["weight_unit"] / 1000.0: -> Ich vermutte mal für deinen Use Case ist es einfacher in Gram einzugeben als in kg, deswegen der Code? Und an weight_unit denke ich "kg" oder "g" und nicht eine Zahl. Ich würde hier auf position["weight_in_kg_net"] ändern
  

  python_dhl_germany/dhl/__init__.py

  Line 254 in 3353e0b

  "netWeightInKG": position["weight_unit"] / 1000.0,

• Die obere Änderung macht dann die Angabe für weight_total konsistent in kg. Ich würde die Variable weight_total in weight_in_kg_total umbenennen
  

  python_dhl_germany/dhl/__init__.py

  Line 209 in 3353e0b

  weightInKG=weight_total

• Passend zu dieser Änderung dann die Anpassung der Zahlen in den Testen, von g zu kg.

Sonst habe ich eine Frage bezüglich is_premium. Ist diese Feature von dem Shipment schon durch das DHL Produkt, zB V53WPAK für DHL Paket International, definiert? Wenn ich das Produkt benutze, wird die Sendung trotzdem Premium wenn ich aus versehen is_premium=False setze?

python_dhl_germany/dhl/__init__.py

Line 198 in 3353e0b

self, dhl_product, dhl_account_number, order_id, weight_total, shipment_date=None, is_premium=False

Viele Grüße
Vinh