fix(params.pac): se cachan las excepciones de suds
MethodNotFound, PortNotFound, ServiceNotFound, TypeNotFound, BuildError, SoapHeadersNotPermitted,WebFault
- Automatically resolved by Cuauhtémoc Díaz Minor
changed the description
- Contributor
@OpenPymeMx revisando el código, me encontré con que no se estaban considerando las excepciones a la hora de comunicarse con los webservices de finkok, se usa la libreria suds que tiene bien definidas las excepciones que lanza al intentar comunicarse con los webservice!
added 2h of time spent at 2018-09-24
¿estas excepciones son las el timbrado de facturas verdad?
- Contributor
@majose.cruz si, son excepciones que se arrojan si falla de alguna manera el uso de los web services de finkok!
resolved all discussions
added 1 commit
- 7c797c26 - fix(params.pac): se cachan las excepciones de suds
- Contributor
@OpenPymeMx los demás que opinan? por un lado tenemos que no se estaban considerando las excepciones que se podían presentar al comunicarse con el webservice de finkok, de ahí salio este MR, y por otro lado tenemos que ya queremos migrar la librería suds que ya tiene mas de 3 años sin mantenimiento, por la de requests! creo que mientras no se haga el cambio de la librería, entonces hacer robusto el código cachando estas excepciones es bueno, ya que cuando se haga el cambio de librería sabremos que hay que considerar las excepciones que se puedan generar con la comunicación con el webservice!
added 10 commits
-
7c797c26...ab258794 - 9 commits from branch
l10n_mx:master
- 7ea5eb6b - fix(params.pac): se cachan las excepciones de suds
-
7c797c26...ab258794 - 9 commits from branch
- Contributor
@OpenPymeMx ping!
- OwnerResolved by Cuauhtémoc Díaz Minor
puedes pasarle el black al archivo para corrija los estilos por favor!
- Last reply by Cuauhtémoc Díaz Minor
- Resolved by Agustín Cruz
added 1 commit
- 5ef76960 - fix(params.pac): se cachan las excepciones de suds
resolved all discussions
- Contributor
@OpenPymeMx ping
- Resolved by Agustín Cruz
resolved all discussions
- Contributor
@OpenPymeMx ping
resolved all discussions
- Owner
Listo, lo mezclamos?
- Contributor
pues si, para seguir avanzando!
added 1 commit
- db150527 - style(black): aplica black a todo el repositorio
merged
- You're only seeing other activity in the feed. To add a comment, switch to one of the following options.