Repository navigation
Conversation
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
This branch was previously deployed
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
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.
Mobile imports can stay in processing after the server has finished. The screen makes four status requests and stops, so a slower conversion's completion or failure never reaches the queue.
Keep polling with the existing delays, capped at eight seconds between requests, until a terminal status arrives. Stop when the item is removed, the screen unmounts, or the signed-in client changes. Check again after an in-flight response so it cannot update a removed item or an old session.
Closes #2945.
Before / after
The same controlled API replies were run through the baseline and patched production functions and actual queue reducer. The terminal reply becomes available at 30 seconds.
Times are virtual schedules with zero simulated request latency, not production conversion measurements. This changes status tracking, not conversion speed.
Validation
git diff --checkpass.@vitest/utils; React Test Renderer is also unavailable. No native-device, real upload, or conversion testing was performed.

Confidence Score: 5/5
The PR appears safe to merge; no actionable issues were found.Summary
Keeps mobile imports tracking server progress beyond the first four checks.
No actionable issues found. This review checked source paths; it did not run the screen tests or capture the native UI.
AksharP5 explicitly describes polling until completion or failure as intentional. Continuing while the server reports processing is the requested behavior.
Reviews (1) · Last reviewed commit: "fix(mobile): keep tracking imports until..." · Reviewed by Greptile