Fix: Surface processing failures to the user instead of spinning#17
Open
Nishit24113 wants to merge 2 commits into
Open
Fix: Surface processing failures to the user instead of spinning#17Nishit24113 wants to merge 2 commits into
Nishit24113 wants to merge 2 commits into
Conversation
The PDF pipeline now writes result/FAILED_<name>.json on any failure (with a reason category and the affected page range). Previously the UI only polled for result/COMPLIANT_<name> and ignored failures, so a failed job left the user watching the spinner until the 30-minute polling timeout. ProcessingContainer now, on each poll (PDF format), checks for the FAILED_ marker before the COMPLIANT check. If found it stops polling and shows a failure card with the summary, affected pages, and reason code, plus a "Try another document" action.
After 30 minutes (120 attempts) the UI now sets a failure state with a clear message instead of clearing the interval with no user feedback.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
result/COMPLIANT_<name>until the 30-minute timeout — user saw a spinner with no feedback.result/FAILED_<name>.jsonfirst. If found, it stops polling immediately and shows a failure card with:Files changed
pdf_ui/src/components/ProcessingContainer.jsx— failure marker polling + failure UI renderingpdf_ui/src/components/ProcessingContainer.css— failure card stylingDepends on
FAILED_<name>.jsonmarker)Test plan