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

[ADD] account_check_printing_caixabank #875

Merged

Conversation

etobella
Copy link
Member

Este módulo permite imprimir cheques con el formato DIN-A-4 de Caixabank. Por desgracia los bancos no se han puesto de acuerdo con los formatos.

@pedrobaeza
Copy link
Member

Caixabank tiene el mismo formato para todo el territorio?

@etobella
Copy link
Member Author

Entiendo que así debe de ser, nosotros tenemos uno que es en catalán y castellano a la vez. Si vemos que la adaptación sólo es local podemos quitar el PR.

@pedrobaeza
Copy link
Member

Es que me suena tener antecedentes de que según la partida de documentos preimpresos que tengan las sucursales, el formato es uno u otro, que como dices, lo ideal es que todos tuvieran el mismo para empezar, pero si no, al menos que fueran homogéneos a lo largo del tiempo/ubicación, pero no sé si podemos concederle ni eso...

Copy link
Sponsor Member

@JordiBForgeFlow JordiBForgeFlow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@etobella etobella force-pushed the 11.0-add-account_check_printing_caixabank branch from ac539fa to 911a505 Compare June 26, 2018 08:01
@JordiBForgeFlow
Copy link
Sponsor Member

@etobella puedes hacer squash de los commits? Esto ya está listo para hacer merge.

@etobella etobella force-pushed the 11.0-add-account_check_printing_caixabank branch from 911a505 to 30c91f5 Compare October 31, 2018 10:29
@JordiBForgeFlow
Copy link
Sponsor Member

@pedrobaeza puedes hacer merge?

@pedrobaeza
Copy link
Member

Faltan los comentarios que puse

@pedrobaeza pedrobaeza added this to the 11.0 milestone Oct 31, 2018
@etobella
Copy link
Member Author

@pedrobaeza, los comentarios ya fueron resueltos en su dia.

@etobella etobella force-pushed the 11.0-add-account_check_printing_caixabank branch from 30c91f5 to b548ca4 Compare February 19, 2019 16:16
@etobella etobella force-pushed the 11.0-add-account_check_printing_caixabank branch from 613921f to ced7f33 Compare March 7, 2019 08:54
@pedrobaeza
Copy link
Member

Revisa Travis

@etobella
Copy link
Member Author

Era un problema del issue de las constrains de odoo

============

In order to install this module you must first install also the module
'report_paper_wkhtmltopdf_params', available in
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esto no es correcto del todo, si no que depende directamente de account_check_printing_report_base

@pedrobaeza
Copy link
Member

Esto está ajustado con la versión de wkhtmltopdf 0.12.5?

@etobella
Copy link
Member Author

Está probado con 0.12.0.4. Probaremos con la 0.12.0.5 a ver que tal.

@pedrobaeza
Copy link
Member

Pues ya te digo que se va a descuanjaringar todo si lo hiciste antes del parche de la DPI que puso Odoo...

@etobella
Copy link
Member Author

Lo sé 😞 , el último commit lo deja correcto con la 0.12.4 con el commit que comentas de odoo

@pedrobaeza
Copy link
Member

Prueba por favor qué tal sale en la 0.12.5 y si sale bien, tiramos para adelante.

@etobella etobella force-pushed the 11.0-add-account_check_printing_caixabank branch from ced7f33 to 6438d62 Compare March 20, 2019 09:23
@etobella
Copy link
Member Author

Ya está testeado con la versión 0.12.5 de wkhtmltopdf y funciona correctamente.

@pedrobaeza pedrobaeza merged commit a308417 into OCA:11.0 Apr 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants