Skip to content

fix: deposit form bug - #2097

Merged
zzacharo merged 2 commits into
CERNDocumentServer:additional-filesfrom
zubeydecivelek:chapters
Aug 27, 2025
Merged

fix: deposit form bug#2097
zzacharo merged 2 commits into
CERNDocumentServer:additional-filesfrom
zubeydecivelek:chapters

Conversation

@zubeydecivelek

Copy link
Copy Markdown
Contributor

Since file_video_extract_chapter_frames is not a main task, it's not in depositStates and that.stateReporter. And it was failing during the status update

that.stateReporter[state].status = value;
}
});
that.calculateCurrentDepositStatus();

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this method will take into account also the chapters task status....Can you check if the chapters task is failed what happens in the UI?

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.

Screenshot 2025-08-27 at 12 06 37

@zubeydecivelek zubeydecivelek Aug 27, 2025

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.

This will remove the chapters task from overall state: d5d85bb

@zzacharo
zzacharo merged commit 32931f4 into CERNDocumentServer:additional-files Aug 27, 2025
1 check passed
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.

2 participants