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 ora submission view filter for turnitin integration #8

Merged
merged 13 commits into from
Feb 7, 2024

Conversation

BryanttV
Copy link
Contributor

@BryanttV BryanttV commented Feb 7, 2024

Description

This PR adds a pipeline step that modifies the ORA submission view rendering process. This step changes the default template by a custom template that has a warning message that notifies the student the Turnitin plagiarism check is enabled.

How to Test

  1. Install this plugin with this branch in the LMS/CMS services.

  2. Install openedx-filters in the LMS/CMS services with these changes

  3. Install edx-ora2 in the LMS/CMS services with these changes

  4. Add the Open edX Filters configuration to your environment. You can use this inline plugin:

    name: filters-settings
    version: 0.1.0
    patches:
      openedx-common-settings: |
        OPEN_EDX_FILTERS_CONFIG = {
          "org.openedx.learning.ora.submission_view.render.started.v1": {
            "fail_silently": False,
            "pipeline": [
              "platform_plugin_turnitin.extensions.filters.ORASubmissionViewTurnitinWarning",
            ]
          },
        }
  5. Restart services

  6. In a unit of the course create a new ora assessment.

  7. In the LMS, in the submission step (1. Your Response), you'll see:

    image

@BryanttV BryanttV force-pushed the bav/ora-submission-view-filter branch from 5ba8e0c to 078d112 Compare February 7, 2024 16:19
Comment on lines 10 to 11
openedx-events @ git+https://github.com/edunext/[email protected]/edues # Open edX event tracking
openedx-filters @ git+https://github.com/edunext/openedx-filters.git@bav/ora-student-view-filter # Open edX filters
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's put a pin on this so we can solve it later: who should manage this kind of dependency? Us while building using tutor patches or the plugin?

@BryanttV BryanttV merged commit 76e2725 into main Feb 7, 2024
5 checks passed
@mariajgrimaldi mariajgrimaldi deleted the bav/ora-submission-view-filter branch April 24, 2024 21:34
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