allegro / allegro-api

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

endpoint /sale/shipping-rates/{id} nie zwraca shippingTime #1747

Closed j4r3kb closed 5 years ago

j4r3kb commented 5 years ago

Występuje problem:

Wywołując tytułowy endpoint dla konkretnego cennika nie otrzymujemy danych odnośnie czasu dostawy mimo iż są ustawione - cennik utworzony w panelu Allegro. Oto fragment cennika:

        {
            "deliveryMethod": {
                "id": "7203cb90-864c-4cda-bf08-dc883f0c78ad"
            },
            "maxQuantityPerPackage": 14,
            "firstItemRate": {
                "amount": "15.99",
                "currency": "PLN"
            },
            "nextItemRate": {
                "amount": "0.00",
                "currency": "PLN"
            },
            "shippingTime": null
        },
MartaNowaczyk commented 5 years ago

@j4r3kb null występuje wtedy, gdy wartość shippingTime jest domyślna. Gdy ustawisz czas ręcznie powinien być widoczny.

j4r3kb commented 5 years ago

Niestety nie, mimo ustawienia na cenniku innych niż domyślne wartości np. 2-5 dn, API dalej zwraca null. Druga sprawa to że null jest domyślny jest słabym pomysłem. Skąd zewnętrzna aplikacja ma wiedzieć co jest domyślne dla danej formy dostawy. Np. Kurier może to być 1 dzień a Paczkomat 2 dni. Użytkownik aplikacji korzystającej z API nie ma żadnej informacji zwrotnej co się ukaże na Allegro po wysłaniu domyślnego null dla danej formy transportu - nie będzie zaglądał na Allegro żeby się dowiedzieć!

j4r3kb commented 5 years ago

@MartaNowaczyk ?

MartaNowaczyk commented 5 years ago

@j4r3kb Zmieniłam u siebie dane i dostaję odpowiednie czasy:

    "id": "8b5d6e96-0772-4f73-bacb-14a944540ae1",
    "name": "drugi - z allegro",
    "rates": [
        {
            "deliveryMethod": {
                "id": "45309171-0415-49cd-b2cf-89e9143d20f0"
            },
            "maxQuantityPerPackage": 1,
            "firstItemRate": {
                "amount": "22.00",
                "currency": "PLN"
            },
            "nextItemRate": {
                "amount": "0.00",
                "currency": "PLN"
            },
            "shippingTime": {
                "from": "PT96H",
                "to": "PT120H"
            }
        },
        {
            "deliveryMethod": {
                "id": "74bc07eb-552f-4581-b68c-da46716d4a9a"
            },
            "maxQuantityPerPackage": 1,
            "firstItemRate": {
                "amount": "14.00",
                "currency": "PLN"
            },
            "nextItemRate": {
                "amount": "0.00",
                "currency": "PLN"
            },
            "shippingTime": null
        },
        {
            "deliveryMethod": {
                "id": "773167b1-feec-4ae9-b20f-1ed8ccb7b1ed"
            },
            "maxQuantityPerPackage": 1,
            "firstItemRate": {
                "amount": "15.00",
                "currency": "PLN"
            },
            "nextItemRate": {
                "amount": "0.00",
                "currency": "PLN"
            },
            "shippingTime": {
                "from": "PT72H",
                "to": "PT168H"
            }
        }
    ],
    "lastModified": "2019-06-06T07:48:23.488Z"
}

Jeśli chodzi o wartości null, to wiemy o takim zapotrzebowaniu i mamy to na uwadze.

j4r3kb commented 5 years ago

Któryś raz z kolei fixujecie problem po zgłoszeniu udając że działało od zawsze, robiąc z ludzi lekko mówiąc łamagi. Dziś to działa zarówno na prod i sandbox, wczoraj (i wcześniej) nie zwracało danych.

MartaNowaczyk commented 5 years ago

Nie zgadzam się z Twoją wypowiedzią. Żadnego fixa nie było po twoim zgłoszeniu.

stale[bot] commented 5 years ago

W tym wątku nie pojawiła się żadna nowa odpowiedź w ciągu 30 dni. Dlatego automatycznie oznaczamy go jako przeterminowany. Jeśli w ciągu 7 dni nie pojawi się żadna odpowiedź, zamkniemy ten wątek. Dziękujemy za zaangażowanie w dyskusję.