PADV 916 - CCX support for Gradebook

[alexjmpb]

Ticket

https://agile-jira.pearson.com/browse/PADV-916

Description

Due to necessity, we found that the gradebook was not working for CCXs after migration. This PR aims to add CCX compatibility to the gradebook.

Changes

• Modify gradebook API authorization to allow CCXs.

How to test?

• Setup devstack
• Checkout to this branch
• Create a CCX (licensed or not)
• Verify the access to the gradebook with a CCX coach, Instructor or Staff user.

Screenshots

[Jacatove]

When you edit course_author_access_required you are not only affecting CourseGradingView, these views are also updated:

• CourseImportView
• CourseQualityView
• CourseValidationView

Might some of them still be important for CCXs?

My initial comment is to extend the platform rather than modify it, given set that, I would recommend creating a new decorator for GradebookView. Then we can discuss the code. @alexjmpb

[Jacatove]

I was conducting this test and the has_user method seems to work:

from common.djangoapps.student.roles import CourseCcxCoachRole, CourseInstructorRole, CourseStaffRole
from django.contrib.auth.models import User
from ccx_keys.locator import CCXLocator

ccx = CCXLocator.from_string('ccx-v1:VUE+1001+2023+ccx@27')

ccx_coach = User.objects.get(email='[email protected]')
instructor = User.objects.get(email='[email protected]') # I added this user to the ccx as admin

CourseCcxCoachRole(ccx).has_user(ccx_coach) # Result is False, since ccx_coach has the role on the master course.
CourseStaffRole(ccx).has_user(coach) # Got True
CourseInstructorRole(ccx).has_user(instructor) # Got True

Besides the has_user working, there might not need to check if a user has the CCX_COACH role on a CCX, since that role is intended only for regular courses and not for CCXs. @alexjmpb