search

allegro / allegro-api

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

Zmiana z WzA #9048

Open jesiiu opened 2 months ago

jesiiu commented 2 months ago

Witam, Mam pytanie odnośnie zmian w API WzA związanych z InPost. Czy od 23 maja będą wymagane zmiany w API? Przykładowo: 2488f7b7-5d1c-4d65-b85c-4cbcf253fd93#id_umowy_wlasnej dla metod Allegro Paczkomaty InPost - oznacza to, że delivery_method_id będzie wyglądać w ten sposób w '/shipment-management/delivery-services' i nie trzeba będzie przekazywać credentials_id do metody generującej etykiety czy trzeba będzie to połączyć tym # przed wysłaniem zapytania?

MartaNowaczyk commented 2 months ago

W /shipment-management/delivery-services zostanie po staremu czyli będzie oddzielnie "deliveryMethodId" i umowa własna w "credentialsId". W zapytaniu musi iść natomiast w odpowiedniej strukturze z #.

jesiiu commented 2 months ago

Czyli jeżeli dotychczas zapytanie wyglądało tak 'input' => [ 'deliveryMethodId' => $deliveryData['delivery_method_id'], 'credentialsId' => $deliveryData['credentials_id'] ?? null,

To od 23 maja dla inpost powinno wyglądać 'input' => [ 'deliveryMethodId' => $deliveryData['delivery_method_id']. '#' . $deliveryData['credentials_id'] ?? null, 'credentialsId' => $deliveryData['credentials_id'] ?? null,

                            Czy w takim razie pole credentialsId jest potrzebne osobno czy można je wyciąć?
                            Po przesłaniu w formie z # rozliczenia będą po stawkach allegro?
MartaNowaczyk commented 2 months ago

Ogólnie sprawa wygląda tak, że dla "nowych" zasobów POST /shipment-management/shipments/create-commands nie możesz pominąć "credentialsId", bo tu mamy osobne pole umowy własnej, czyli w "deliveryMethodId" przekazujesz id usługi, a w "credentialsId" id umowy własnej. Natomiast w starym zasobie używasz połączenia obu wartości za pomocą #. Ogólnie w API nic się nie zmienia.

jesiiu commented 2 months ago

Czyli jeżeli moje zapytanie wygląda w ten sposób 'input' => [ 'deliveryMethodId' => $deliveryData['delivery_method_id'], 'credentialsId' => $deliveryData['credentials_id'] ?? null,

To nic nie muszę zmieniać i będzie prawidłowo rozliczone po 23 maja, tak?

AureliuszBrussy commented 2 months ago

Tak, będzie to prawidłowe - nie musisz nic zmieniać.

jesiiu commented 1 month ago

Oczywiście prawidłowo rozliczone to mam na myśli, że rozliczone po stawkach allegro, tak?

MartaNowaczyk commented 1 month ago

Tak

jesiiu commented 1 month ago

{ "delivery_method_id":"321...", "credentials_id":"123..", "delivery_name":"Kurier InPost (inpost1)" }, { "delivery_method_id":"321..", "credentials_id":"123..", "delivery_name":"Kurier InPost pobranie (inpost1)" }, { "delivery_method_id":"321..", "credentials_id":"123..", "delivery_name":"Paczkomaty InPost (inpost1)" }, { "delivery_method_id":"321..", "credentials_id":"123..", "delivery_name":"Paczkomaty InPost pobranie (inpost1)" },

Wyświetliłem sobie dane z zasobu "/shipment-management/delivery-services" i wygląda to mniej więcej w ten sposób, dokładnie tak samo jak przed "zmianami" z InPost, prosiłbym tylko o potwierdzenie, że wszystko jest poprawnie i przesyłki będą rozliczane przez stawki allegro. ID są zmienione przeze mnie, nigdzie nie było znaku '#' w id

MartaNowaczyk commented 1 month ago

tak, nie będzie tam # w ID. Zmiany o których pisaliśmy dotyczą metod z Allegro na początku:

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.