Skip to content

print job status polling #1894

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 23 commits into
base: dev
Choose a base branch
from
Open

print job status polling #1894

wants to merge 23 commits into from

Conversation

@thebeninator
Copy link
Collaborator

@thebeninator thebeninator commented Aug 29, 2025 •
edited
Loading

resolves #1857

@thebeninator
save
57bf38d
@thebeninator
poll job status on print page; local storage first pass
890a5b3
@thebeninator
delete printId from local storage on complete
6add54c
@thebeninator
resume polling if printId still exists
8f50098
@thebeninator
add undefined check for localStorage in useEffect
d8179ce
@thebeninator
save
aad6860
@thebeninator
save
5403349
@thebeninator
use promise for tryResolvePrint
de4e720
@thebeninator
convert /status to GET, require token
177fc20
@thebeninator
multiple print job tracking first pass
60ad976
@thebeninator
fix lint
9eb41ee
@thebeninator
fix interval using stale jobs state
78d7f23
@thebeninator
implement doodoo ui for active prints
837b980
@thebeninator
store job in local immediately; remove guard clause for 'PRINTED' state
eb764ad
@thebeninator
some clean up
ae994b7
@thebeninator
FIX LINT
0c0767a
@thebeninator
check if localstorage printJobs exists first before setting state
2f15f17
@thebeninator
undoodoo the doodoo ui first pass
7faa2bc
@thebeninator
implement page escrow
6ab42d1
@thebeninator
some cleanup; actually save escrow modify on print fail
85a6671
@thebeninator
failed/completed jobs notifs now persist for 5 secs
02b2748
evanugarte
evanugarte reviewed Nov 19, 2025
View reviewed changes
Copy link
Collaborator

@evanugarte evanugarte left a comment

Choose a reason for hiding this comment

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

lets test this out on the real website when possible

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@evanugarte evanugarte evanugarte left review comments

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

print job status polling

3 participants

@thebeninator @evanugarte