allegro / allegro-api

Issue tracker and wiki for Allegro REST API
https://developer.allegro.pl/
212 stars 39 forks source link

PATCH /sale/product-offers i "Missing required parameters" #9432

Open tomylee0 opened 1 month ago

tomylee0 commented 1 month ago

Nie jestem pewien czy dobrze zrozumiałem #9193 Bo np. tutaj przy PATCH'u /sale/product-offers otrzymuję błąd: Missing required parameters: 235009 (trace-id: 721e96872477122b)

A ten fragment z #9193 mówi mi, że i ilość sztuk i cenę można zmieniać nawet przy braku obowiązkowego parametru:

Oznacza to, że - podobnie jak w przypadku zmiany liczby sztuk - nawet jeśli w ofercie będzie brakować wymaganego parametru, to samą operację zmiany ceny przeprowadzimy z pozytywnym rezultatem.

MartaNowaczyk commented 1 month ago

Tak, ale dzieje się tak w przypadku zmiany samej liczby sztuk, a nie w przypadku podania jeszcze innych pól. W podanym przykładzie oprócz stocku zmieniasz jeszcze "taxSettings".

tomylee0 commented 1 month ago

Ok. W tym przypadku tak rzeczywiście było.

Skoro jednak można zmieniać ilość lub cenę nawet gdy nie ma obowiązkowego parametru to mam wątpliwości co do zabraniania modyfikacji taxSettings. TaxSettings jest mocno powiązany z ceną. Czasem ktoś wpisze złą stawkę VAT'u. Poprawienie stawki VAT wymusza poprawę ceny w ofercie. I tu spotkamy się z blokadą. Jeśli więc można zmieniać ilość i cenę to może nie byłby to problem, żeby również taxSettings można było zmieniać?

Pozdrawiam.

MartaNowaczyk commented 1 month ago

W taki sposób można wiele rzeczy/pól połączyć i każdy mógłby chcieć co innego. W każdym razie dwa kluczowe parametry oferty można zmienić, a na tym nam zależało. Pamiętaj, że zawsze możesz dodać swoją sugestię do zakładki Discussions. Pozdrawiam

stale[bot] commented 1 month ago

W tym wątku nie pojawiła się żadna nowa odpowiedź w ciągu 7 dni, dlatego automatycznie oznaczamy go jako przeterminowany. Jeśli w ciągu kolejnych 7 dni nie pojawi się żadna odpowiedź, wątek zostanie zamknięty. Dziękujemy za zaangażowanie w dyskusję i zachęcamy Cię do wypełnienia ankiety dotyczącej naszego wsparcia na forum.


There was no new reply in this thread within 7 days, therefore, we automatically marked it as expired. If no response is received within the next 7 days, this thread will be closed. Thank you for engaging in the discussion. We encourage you to complete our forum support survey.