fix: update Zoom live configurations to support email for course staff roles. (#31257)

Co-authored-by: SaadYousaf <saadyousaf@A006-00314.local>
This commit is contained in:
Saad Yousaf
2022-11-15 18:38:46 +05:00
committed by GitHub
parent 5aea84f1df
commit eb8f3b6750
3 changed files with 17 additions and 3 deletions

View File

@@ -87,9 +87,6 @@ class Zoom(LiveProvider):
"""
id = 'zoom'
name = 'Zoom LTI PRO'
additional_parameters = [
'custom_instructor_email'
]
@property
def is_enabled(self):

View File

@@ -4,6 +4,8 @@ Configurations to render Course Live Tab
from django.utils.translation import gettext_lazy
from lti_consumer.models import LtiConfiguration
from common.djangoapps.student.roles import CourseStaffRole, CourseInstructorRole
from xmodule.course_module import CourseBlock
from xmodule.tabs import TabFragmentViewMixin
from lms.djangoapps.courseware.tabs import EnrolledTab
@@ -69,3 +71,17 @@ class CourseLiveTab(LtiCourseLaunchMixin, TabFragmentViewMixin, EnrolledTab):
super().is_enabled(course, user) and
CourseLiveConfiguration.is_enabled(course.id)
)
def _get_pii_lti_parameters(self, course, request):
pii_config = super()._get_pii_lti_parameters(course, request)
provider_type = ''
course_live_configurations = CourseLiveConfiguration.get(course.id)
if course_live_configurations:
provider_type = course_live_configurations.provider_type
if provider_type == 'zoom' and (CourseStaffRole(course.id).has_user(request.user) or
CourseInstructorRole(course.id).has_user(request.user)):
pii_config['person_contact_email_primary'] = request.user.email
return pii_config

View File

@@ -57,6 +57,7 @@ class LtiCourseLaunchMixin:
pii_config = {}
if lti_config.pii_share_username:
pii_config['person_sourcedid'] = request.user.username
if lti_config.pii_share_email:
pii_config['person_contact_email_primary'] = request.user.email
return pii_config