Skip to content

send only allowed file types to iThenticate#97

Open
forgive38 wants to merge 1 commit into
pkp:stable-3_4_0from
forgive38:allowed_file_types
Open

send only allowed file types to iThenticate#97
forgive38 wants to merge 1 commit into
pkp:stable-3_4_0from
forgive38:allowed_file_types

Conversation

@forgive38

Copy link
Copy Markdown
Contributor

Hello

I propose this PR in order to restrict the number of files sent to iThenticate, based on iThenticate allowed file types.

plagiat

I hope that helps,
Thank you

@asmecher

asmecher commented Jan 9, 2026

Copy link
Copy Markdown
Member

@forgive38, just curious -- what happens when an unsupported file type is uploaded to the iThenticate service?

@forgive38

Copy link
Copy Markdown
Contributor Author

@asmecher : there is an error when you schedule the detection

For example, screenshot for the file J.png
ErrorBadFormat

And since all files of a new submission are automatically sent to iThenticate, that adds an extra delay: it impacts the ui experience (sending files to iThenticate is synchronous, maybe it can be asynchronous via a Job ? - I will make an issue)

@touhidurabir

Copy link
Copy Markdown
Member

maybe it can be asynchronous via a Job

@forgive38 I think thats a good idea. but as that going to some good amount of enhancement(probably), I may suggest to go for such major change in 3.5+ compatible only . But a separate issue is a good place to discuss about that .

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.

3 participants