Skip to content

Commit a33ba23

Browse files
authored
Merge pull request #386 from shrinishLT/DOT-6317
fix approved status
2 parents 0398b80 + a35a013 commit a33ba23

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

src/lib/constants.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -126,6 +126,7 @@ export default {
126126
MOBILE_ORIENTATION_LANDSCAPE: 'landscape',
127127

128128
// build status
129+
BUILD_RUNNING: 'running',
129130
BUILD_COMPLETE: 'completed',
130131
BUILD_ERROR: 'error',
131132

src/lib/utils.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -545,7 +545,7 @@ export function startPdfPolling(ctx: Context) {
545545
try {
546546
const response = await ctx.client.fetchPdfResults(ctx);
547547

548-
if (response.screenshots && response.build?.build_status === constants.BUILD_COMPLETE) {
548+
if (response.screenshots && response.build?.build_status !== constants.BUILD_RUNNING) {
549549
clearInterval(interval);
550550

551551
const pdfGroups = groupScreenshotsByPdf(response.screenshots);

0 commit comments

Comments
 (0)