Skip to content

add fishpaper and printer status query#287

Merged
BNH440 merged 6 commits into
masterfrom
printer-stats-update
May 6, 2026
Merged

add fishpaper and printer status query#287
BNH440 merged 6 commits into
masterfrom
printer-stats-update

Conversation

@BNH440
Copy link
Copy Markdown
Contributor

@BNH440 BNH440 commented May 6, 2026

adds fishpaper to list of printers and adds a way to query the printer status string via snmp (gets data like if printer is out of paper)

jaysa68
jaysa68 previously approved these changes May 6, 2026
Copy link
Copy Markdown
Member

@jaysa68 jaysa68 left a comment

Choose a reason for hiding this comment

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

rebase on main and ensure tests pass + it builds, then you are good to go!

@BNH440 BNH440 marked this pull request as draft May 6, 2026 04:29
@BNH440
Copy link
Copy Markdown
Contributor Author

BNH440 commented May 6, 2026

going to add paper tray status too

@BNH440 BNH440 force-pushed the printer-stats-update branch from e2837b3 to 45f2b16 Compare May 6, 2026 04:30
@BNH440 BNH440 marked this pull request as ready for review May 6, 2026 04:51
@BNH440 BNH440 requested a review from jaysa68 May 6, 2026 04:51
import puresnmp

PRINTERS = ['logjam', 'pagefault', 'papercut']
PRINTERS = ['logjam', 'pagefault', 'papercut', 'fishpaper']
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

can we query ldap and get printers in prod env

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

that would be nice but probably not realistic right now as we'd also need to tell the difference between hp printers and not

@BNH440 BNH440 merged commit 955b605 into master May 6, 2026
1 check passed
@BNH440 BNH440 deleted the printer-stats-update branch May 6, 2026 04:56
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