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 open responses card to page and resources #32589

Merged
merged 1 commit into from
Jul 13, 2023

Conversation

leangseu-edx
Copy link
Contributor

@leangseu-edx leangseu-edx commented Jun 27, 2023

Description

Adding Open Responses to turn on flexible peer grading for the course.

Screenshot 2023-07-10 at 4 52 31 PM Screenshot 2023-07-10 at 4 52 24 PM

Ticket: https://2u-internal.atlassian.net/browse/AU-1279
Related PR: openedx/frontend-app-authoring#524

Useful information

  • It seems the plugin is using course_apps_courseappstatus for the settings. There isn't the need to update the model.
  • is_enabled seem to be use as default unless there is an entry in course_apps_courseappstatus
  • You can query this config with CourseAppStatus.objects.get(course_key=course_key, app_id='flexible_peer_grading')

Testing instructions

  • You need to enable course waffle flag discussions.pages_and_resources_mfe to have Page & Resource option on the course outline.
  • You should be able to see Open response options
    • If the option isn't available you might need to
      • go to devstack make dev.shell.studio
      • make requirements for setup.py to pick up
      • then restart lms + studio with make lms-restart studio-restart or make dev.restart-container.lms+studio if the restart didn't work.

lms/djangoapps/courseware/plugins.py Outdated Show resolved Hide resolved
lms/djangoapps/courseware/plugins.py Outdated Show resolved Hide resolved
lms/djangoapps/courseware/plugins.py Outdated Show resolved Hide resolved
lms/envs/common.py Outdated Show resolved Hide resolved
@leangseu-edx leangseu-edx force-pushed the lk/add-ora-setting-course-app branch from a37d4ca to 348cc0f Compare July 10, 2023 21:08
@leangseu-edx leangseu-edx merged commit aa987c3 into master Jul 13, 2023
43 checks passed
@leangseu-edx leangseu-edx deleted the lk/add-ora-setting-course-app branch July 13, 2023 14:35
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

1 similar comment
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

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.

4 participants