Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wysyłam z Allegro - błąd pól pickup.pointId i label.sender.pointId #7613

Closed
btc-jz opened this issue Sep 12, 2023 · 20 comments
Closed

Wysyłam z Allegro - błąd pól pickup.pointId i label.sender.pointId #7613

btc-jz opened this issue Sep 12, 2023 · 20 comments

Comments

@btc-jz
Copy link

btc-jz commented Sep 12, 2023

Mam następujący problem, dziś przy próbach utworzenia przesyłki do Allegro Paczkomat (PUT do /parcel-management/parcel-create-commands/) zacząłem otrzymywać komunikat o błędach walidacji:

VALIDATION_ERROR :: pickup.pointId musi pasować do wyrażenia [A-Za-z0-9-]+.
VALIDATION_ERROR :: label.sender.pointId musi pasować do wyrażenia [A-Za-z0-9-
]+.

W dokumentacji (https://developer.allegro.pl/documentation/#operation/createNewParcel) pole pickup.pointId nie jest obowiązkowe, pole label.sender.pointId w ogóle nie jest wymienione. Czy dokumentacja jest nieaktualna?

@PrzemyslawLukanowski
Copy link
Collaborator

Czy możesz podać trace-id lub commandId, dla których otrzymałeś wymienione błędy?

@btc-jz
Copy link
Author

btc-jz commented Sep 12, 2023

Przepraszam, źle opisałem problem, błąd dostaję w odpowiedzi na GET /parcel-management/parcel-create-commands/ wywołanym po utworzeniu przesyłki, przykładowe CommandId to e7241034-e7e9-458d-9ef7-8385d4ca6caf

@CF-WR617AC
Copy link

Potwierdzam, u nas też pojawił się od godziny ten błąd, nie ma póki co rozwiązania

@CF-WR617AC
Copy link

Rano chodziło OK, teraz pluje błędem:
musi pasować do wyrażenia [A-Za-z0-9-_]+ ścieżka: receiver.pointId detale: Invalid value:

@CF-WR617AC
Copy link

Poprzez Allegro na te same dane naklejki wychodzą po WzA. Problem jest z usługą "INPOST minikurier24"

@SoDaUser
Copy link

Prośba o pomoc, dziś normalnie paczki wychodziły obecnie jest błąd jak wyżej opisany:

https://api.allegro.pl/parcel-management/parcel-create-commands/a9d34571-3bff-4aee-aef4-69b483f1f5a8

zwraca błąd:

{"id":"a9d34571-3bff-4aee-aef4-69b483f1f5a8","parcelId":null,"status":"ERROR","errors":[{"code":"VALIDATION_ERROR","message":"musi pasować do wyrażenia [A-Za-z0-9-]+","details":"Invalid value: ","path":"pickup.pointId","userMessage":"musi pasować do wyrażenia [A-Za-z0-9-]+"},{"code":"VALIDATION_ERROR","message":"musi pasować do wyrażenia [A-Za-z0-9-]+","details":"Invalid value: ","path":"label.sender.pointId","userMessage":"musi pasować do wyrażenia [A-Za-z0-9-]+"}]}

@PrzemyslawLukanowski
Copy link
Collaborator

Komunikat błędu jest związany z wprowadzeniem nowej walidacji - nie przyjmujemy już pustego stringa "", przekazujcie zamiast tego proszę wartość null. Przepraszam za te niedogodności.

Co do pola label.sender.pointId, które nie występuje w requeście oraz w dokumentacji - weryfikuję jeszcze ten temat.

@btc-jz
Copy link
Author

btc-jz commented Sep 12, 2023

Nie wiem czy ktoś coś już zmienił w temacie pola label.sender.pointId, ale po podaniu null w pickup.pointId paczka się utworzyła.

@SoDaUser
Copy link

U mnie też poszło na testowej przesyłce po ustawieniu pickup.pointId null.

Korzystamy z aplikacji, ta zmiana to wprowadzenie poprawki, instalki, i aktualizacji.
Mamy szczyt sezonu :/

@SoDaUser
Copy link

@btc-jz ta sama walidacja wprowadzona jest w receiver.pointId w przypadku gdy jest kurier.

W temacie walidacji warto poprawić aby:
w polu ulicy nie wpisywać informacji, że płot jest kamienny, budynek zielony i wejście od tyłu
w polu ulicynie wpisywać miejscowości i kodu pocztowego
aby nie robic tego podwójnie

Wydaje mi się, że często powinno być uzyte pole uwagi

telefon też często ma postać +XXX XXX XXX, 0XXX XXX XXX , XXX-XXX-XXX

@q3ok
Copy link

q3ok commented Sep 13, 2023

Allegro Kurier DPD -błąd z receiver.pointID, przekazanie wartosci pustej, null, 0, 1 nie działą i ciągle zwraca błąd VALIDATION ERROR, jako "punkt odbiorcy jest niepoprawny". Co mam przekazac w pointID?
uuid przykładowy: 39bdb8d2-6f7e-49cf-a79c-272a176c2e9d

@Onixarts
Copy link

Komunikat błędu jest związany z wprowadzeniem nowej walidacji - nie przyjmujemy już pustego stringa "", przekazujcie zamiast tego proszę wartość null. Przepraszam za te niedogodności.

Ale czy to nie są łamiące zmiany? Nie da się od tak u wszystkich użytkowników zmienić sposób walidacji. Prośba o przywrócenie poprzedniego działania bo klienci nie mogą generować etykiet. A w przyszłości takie zmiany powinny być ogłaszane wcześniej, bo one są równie istotne co zmiany w API.

@Zbigniew-L
Copy link

Komunikat błędu jest związany z wprowadzeniem nowej walidacji - nie przyjmujemy już pustego stringa "", przekazujcie zamiast tego proszę wartość null. Przepraszam za te niedogodności.

Ale czy to nie są łamiące zmiany? Nie da się od tak u wszystkich użytkowników zmienić sposób walidacji. Prośba o przywrócenie poprzedniego działania bo klienci nie mogą generować etykiet. A w przyszłości takie zmiany powinny być ogłaszane wcześniej, bo one są równie istotne co zmiany w API.

Popieram kolegę powyżej, dodatkowo chciałbym zauważyć że w dokumentacji createNewparcel NIE MA nawet pola pointid w label.sender... po prostu zrobiliście niespodziankę dla wielu osób, czy dodacie to pole do dokumentacji czy inne rozwiązanie? na razie prosimy o przywrócenie systemu aby działał jak wcześniej... jest na to szansa?
label sender

@SoDaUser
Copy link

Robiłem szybka poprawkę dla box, drugą dla kuriera,
tu czytam, że z dpd też będa problemy.

W okresie zakupów szkolnych każda minuta jest dla nas na wagę złota.
Może uda się, przywrócić poprzednią walidację, aby ewentualne niespodzianki można bezpiecznie/spokojnie wdrożyć ?

@PrzemyslawLukanowski
Copy link
Collaborator

Do godziny 12:00 wprowadzimy poprawkę, która będzie zmieniać "" na null, jeszcze raz przepraszam za niedogodności.

@q3ok, @Zbigniew-L - wrócę jeszcze z odpowiedzią co do Waszych pytań.

@PrzemyslawLukanowski
Copy link
Collaborator

@q3ok W Twoim przypadku problemem jest to, że wartość null przekazujesz "opakowaną" w cudzysłów, przez co traktujemy ją jako wartość stringową.

@q3ok
Copy link

q3ok commented Sep 13, 2023

@q3ok W Twoim przypadku problemem jest to, że wartość null przekazujesz "opakowaną" w cudzysłów, przez co traktujemy ją jako wartość stringową.

tak, nie jestem w stanie zrobić innej zmiany na szybko. liczę, że zmiana "" na null będzie działać poprawnie, bo to jest do wdrożenia na szybko.
dlaczego nie można robić tak jak było dotychczas, czyli przy przesyłkach kurierskich pomijać te pole?

@PrzemyslawLukanowski
Copy link
Collaborator

PrzemyslawLukanowski commented Sep 13, 2023

Poprawka jest już na produkcji, problem nie powinien występować.

@Zbigniew-L W tym przypadku, komunikat o label.sender.pointId wprowadza w błąd, ponieważ w rzeczywistości dotyczy on także pola pickup.pointId. Ze względu na nasze wewnętrzne zależności, na tę chwile nie możemy go niestety usunąć.

@q3ok Dotychczas akceptowaliśmy pusty string "", właśnie wdrożyliśmy poprawkę, która nadal to umożliwia, a dalej - po naszej stronie - zamieniamy go na null, dzięki czemu nie powinniśmy odpowiadać komunikatem błędu.

@Onixarts
Copy link

Dzięki, mamy potwierdzenie od użytkowników, że jest już OK.

@stale
Copy link

stale bot commented Sep 21, 2023

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.

@stale stale bot closed this as completed Sep 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants