infifni / fan-courier-api-client

MIT License
13 stars 6 forks source link

Marcare field plata_ramburs_la ca non-required #3

Closed pintilei closed 1 year ago

pintilei commented 1 year ago

Salut,

Vreau sa va multumesc pentru acest package, imi scuteste foarte multa munca si este foarte inteligent gandit!

Implementez o aplicatie iar clientul livreaza coletele achitate integral de catre el si nu solicita ramburs, drept urmare, pe awb-ul generat nu ar trebui sa apara nimic in cadrul sectiunii PLATA TRANSPORT / RAMBURS / RESTITUIRE.

Am incerca sa pun null pentru fieldul "plata_ramburs_la" insa este required.

vendor/infifni/fan-courier-api-client/src/Request/GenerateAwb.php:149, l-am marcat ca required => false si totul este in regula, deci cred ca ar trebui luata in calcul si aceasta optiune.

Documentatie FAN Courier: Plata rambursului la: se completeaza numai daca se solicita ramburs, adica in cazul in care ati completat campul ramburs (completati cu expeditor sau destinatar);

PR daca totul este ok: #4

Screenshot 2023-04-28 at 10 35 00 PM

Dupa modificare:

Screenshot 2023-04-28 at 10 43 39 PM

Multumesc si mult succes, Adrian

laurentiu-cocos commented 1 year ago

Salut,

Ma bucur ca l-ati gasit util.

Am sa ma uit peste PR si il merge-ui, va trebui sa folositi la versiune dev-master in composer pana cand apuc sa fac un release.

Success si toate cele bune, Laurentiu

On Fri, 28 Apr 2023 at 22:42, Adrian Pintilei @.***> wrote:

Salut,

Vreau sa va multumesc pentru acest package, imi scuteste foarte multa munca si este foarte inteligent gandit!

Implementez o aplicatie iar clientul livreaza coletele achitate integral de catre el si nu solicita ramburs, drept urmare, pe awb-ul generat nu ar trebui sa apara nimic in cadrul sectiunii PLATA TRANSPORT / RAMBURS / RESTITUIRE.

Am incerca sa pun null pentru fieldul "plata_ramburs_la" insa este required.

vendor/infifni/fan-courier-api-client/src/Request/GenerateAwb.php:149, l-am marcat ca si false si totul este in regula, deci cred ca ar trebui luata in calcul si aceasta optiune.

[image: Screenshot 2023-04-28 at 10 35 00 PM] https://user-images.githubusercontent.com/11697496/235237313-d5e583d6-3e18-41b3-b5b3-c47eac1962de.png

Multumesc si mult succes, Adrian

— Reply to this email directly, view it on GitHub https://github.com/infifni/fan-courier-api-client/issues/3, or unsubscribe https://github.com/notifications/unsubscribe-auth/AORFOAIX4S7K3B3ABZUXFIDXDQMUBANCNFSM6AAAAAAXPUJRPI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- Laurenţiu Cocoş Web software engineer Administrator of S.C. Infifni Dezvoltare Software S.R.L. https://infifnisoftware.ro/

laurentiu-cocos commented 1 year ago

Trebuie redeschis PR-ul si atunci pot sa-i dau merge.

pintilei commented 1 year ago

Salut,

Multumesc pentru raspuns. Am schimbat eu denumirea branch-ului pentru a-l adauga in composer ca sa continuam dezvoltarea si aparent nu mai pot sa-l redeschid, asa ca am facut unul nou:

https://github.com/infifni/fan-courier-api-client/pull/5

Multumesc, Adrian