Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add flag to enable turnitin submission #18

Merged
merged 11 commits into from
May 9, 2024

Conversation

BryanttV
Copy link
Contributor

@BryanttV BryanttV commented May 7, 2024

Description

This PR adds the possibility to activate the Turnitin functionality using two types of configuration:

  • Globally using a Django settings (ENABLE_TURNITIN_SUBMISSION)
  • At course level using other_course_settings (ENABLE_TURNITIN_SUBMISSION)

How to Test

  1. Install this plugin with the changes in this branch in your environment.
  2. Install edx-ora2, openedx-events, and openedx-filters with the latest versions.
  3. In a unit of the course create a new ora assessment, maybe Staff Assessment Only.
  4. In the settings of the component > File Uploads Response = Required and in File Types = pdf,doc,docx
  5. By default, the functionality is disabled, therefore, no warning message will be displayed, nor will the files be sent to Turnitin.
  6. If activated using either of the two settings mentioned above, a warning message will be displayed, and the files will be sent to Turnitin.

@BryanttV BryanttV marked this pull request as ready for review May 8, 2024 21:30
platform_plugin_turnitin/extensions/filters.py Outdated Show resolved Hide resolved
platform_plugin_turnitin/handlers.py Outdated Show resolved Hide resolved
@BryanttV BryanttV force-pushed the bav/add-flag-for-turnitin-submission branch 2 times, most recently from 71fa6c6 to f60c945 Compare May 9, 2024 18:24
@BryanttV BryanttV force-pushed the bav/add-flag-for-turnitin-submission branch from f60c945 to 9d83d96 Compare May 9, 2024 18:25
@BryanttV BryanttV merged commit 50d6c95 into main May 9, 2024
5 checks 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