Skip to content
This repository has been archived by the owner on Jul 22, 2020. It is now read-only.

Se implementa lectura básica de impuestos trasladados de un XML. #2

Closed
wants to merge 1 commit into from

Conversation

lcarrasco
Copy link

No description provided.

@unRob
Copy link
Owner

unRob commented Jul 29, 2014

Gracias por el PR Luis! Puse un comentario en el PR porque creo que una clase para impuestos puede ser buena idea, pero como no le entiendo muy bien al Anexo 20, no sabría como comenzar a implementar.

Por otro lado, no pasaron los builds por el test que espera que el parsing sea idéntico a una estructura pre-armada, le puedes echar un ojo?

@unRob
Copy link
Owner

unRob commented Jul 29, 2014

Ya vi el test, no creo que haya problema con esto, sólo hay que cambiar la línea 44 de test/comprobante_spec.rb de :impuestos=>[{:impuesto=>"IVA"}] a :impuestos=>[{:impuesto=>"IVA", :tasa=>16.0, :importe=>1760.0}], pero antes del merge, me gustaría ver que onda con la clase de impuestos.

@lcarrasco
Copy link
Author

Que tal Rob buen día, tienes razón anoche ya era un poco tarde y se me pasó crear alguna prueba relacionada. Déjame pienso alguna base de la clase Impuesto que siga respetando la simplicidad de tu código.

P.D. Muchas gracias por publicar tu código, está muy bien hecha. Saludos!

@unRob unRob closed this Jul 1, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants