-
Notifications
You must be signed in to change notification settings - Fork 194
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
[WIP] feat: add ORA filter for student_view rendering intervention #2000
Conversation
Thanks for the pull request, @mariajgrimaldi! Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. This is currently a draft pull request. When it is ready for our review and all tests are green, click "Ready for Review", or remove "WIP" from the title, as appropriate. |
Thanks for the contribution @mariajgrimaldi! :) |
Hey @mariajgrimaldi, just checking in to see if you're still planning to continue working on this PR? |
@itsjeyd: thanks for reaching out! I'll close this for the time being and re-open it when needed. |
@mariajgrimaldi Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future. |
1 similar comment
@mariajgrimaldi Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future. |
@mariajgrimaldi Sounds good, thanks 🙂 |
TL;DR - Add an Open edX Filter for
student_view
This PR adds a new extension point for the
student_view
handler. By triggering an Open edX Filter before rendering the block'sstudent_view
we could:What motivates us to add this filter is this case specifically:
As a professor or student, I want to check my essay with an external service like Turnitin for plagiarism.
More on the filter's definition in this PR: openedx/openedx-filters#107
What changed?
openedx-filters
to the requirements listDeveloper Checklist
Testing Instructions
This is a proof of concept, so testing it can be wacky. Here's a list of steps you can follow so you can test the ongoing implementation:
Reviewer Checklist
Collectively, these should be completed by reviewers of this PR: