The patch

CommentFileSizeAuthor
#3 GUIACOMERCIOSSISv5.19.pdf1.79 MBfacine
#3 commerce_sermepa-add_missing_response_errors-02.patch1.85 KBfacine
commerce_sermepa-add_missing_response_errors.patch2.09 KBfacine

Comments

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Needs review » Needs work

I'm not sure these are all codes available, we probably want to expose this in its very own file as it seems that the error codes are many: http://www.catalunyacaixa.com/docsdlv/Portal/Ficheros/Pdf/es/tpv_virtual...

I'm also not sure if those return codes are standard or each bank has their own.

plopesc’s picture

plopesc
he/him
Primary language Spanish
commented

Hello
In my opinion, the error codes are standard, because depends on the TPV service provided by sermepa. Error codes for other banks, like Banc Sabadell or Deutsche Bank are the same as described in the comment above.

Maybe the error codes could be integrated in the module in order to provide better support for site managers.

Regards.

facine’s picture

facine commented
StatusFileSize
new commerce_sermepa-add_missing_response_errors-02.patch1.85 KB
new GUIACOMERCIOSSISv5.19.pdf1.79 MB

Attached patch with error codes, according to the official documentation.

Also attached official documentation

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Needs work » Needs review
pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Needs review » Fixed

Fixed, thanks both!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

candelas’s picture

candelas commented
Issue summary: View changes

@facine he traducido todas las variables de tu patch a copiando y pegando lo que da el banco en cada error. Ahora faltaria que alguien lo moderase para aceptarlo. Gracias por el patch que ya está en la versión estable :)