Skip to content

await driver methods#278

Open
ssxv wants to merge 3 commits intoKlemen1337:masterfrom
ssxv:master
Open

await driver methods#278
ssxv wants to merge 3 commits intoKlemen1337:masterfrom
ssxv:master

Conversation

@ssxv
Copy link
Contributor

@ssxv ssxv commented Feb 8, 2026

For USB connected printers,

  • await driver.getPrinter in isPrinterConnected method.
  • await driver.getPrinters in getPrinterName
  • update printer retrieval in isPrinterConnected and execute methods

@andidev
Copy link

andidev commented Feb 8, 2026

@ssxv I was about to create similar pr. I would suggest to add following to this PR:
ssxv/node-thermal-printer@master...andidev:node-thermal-printer:master

current references in docs to node-printer libs are just too outdated and does not work with latest javascript so worth changing those as well.

@Klemen1337 what do you say?

…trieval in isPrinterConnected and execute methods
@ssxv ssxv changed the title await driver.getPrinter in isPrinterConnected method await driver methods Feb 14, 2026
@andidev
Copy link

andidev commented Feb 15, 2026

@ssxv I think the Interface class


should also be changed to async.
Seems like it’s only used for the printer driver implementation so should be safe.
Though @Klemen1337 this is a smaller breaking change since now it will return a promise, so probably worth mentioning in the release notes.

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.

2 participants