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

[NEW] Printer with cups #24

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mileo
Copy link
Contributor

@mileo mileo commented Apr 14, 2022

  1. Create a temp file
  2. Send it to cups server.

@mileo
Copy link
Contributor Author

mileo commented Apr 16, 2022

@danielgoncalves o uso do cups como administrador da fila de impressão se mostrou promissor para evitar problemas de concorrência. Quando múltiplas estações enviam impressões para a mesma impressora. Apesar da utilização dos comandos de status da impressora.

cc: @lfdivino @luismalta

Copy link
Contributor

@danielgoncalves danielgoncalves left a comment

Choose a reason for hiding this comment

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

Parece perfeito, com exceção do detalhe da documentação que eu apontei no código. Obrigado pela contribuição :)

@classmethod
def create(cls, setting, **kwargs):
"""Instantiate a :class:`NetworkConnection` (or subclass) object based
on a given host name and port number (eg. ``192.168.0.205:9100``).
Copy link
Contributor

Choose a reason for hiding this comment

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

Provavelmente a documentação precisa ser ajustada, pois parece não coincidir com a implementação. No código, o separador é uma vírgula. Poderia ajustar, por favor? :)

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.

None yet

3 participants