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

Nombre de archivos y funciones nuevas #1

Open
wants to merge 4 commits into
base: trunk
Choose a base branch
from

Conversation

juanlb
Copy link

@juanlb juanlb commented Aug 5, 2014

Corregí los nombres de los archivos para que sea más claro.
También saqué los caracteres no ascii en los mensajes, porque daba errores en el envío por ajax.
Agregué la función getSubtotal, getErrors y getPrinterId
La funcion getPrinterVersion está comentada porque solo sirve para la 320
Agregué la posibilidad de poner datos de impresion de header al crear una factura.

@reingart
Copy link
Owner

Perdón por la demora, justo estoy terminando mi proyecto de GSoC2014 y no pude verlo en detalle.

En principio coincido en que hace falta el renombramiento, pero creo que se puede simplificar y unificar aún más para tener más consistencia:

  • epsonFiscal.py -> epson.py
  • epsonFiscalDriver.py -> driver.py
  • epsonFiscalDriver_service.py -> service.py
  • fiscalGeneric.py -> generic.py
  • hasarPrinter.py -> hasar.py

¿Opiniones?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants