Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: Hide lti_configuration from non-LTI discussions providers

Browse Source
This commit is contained in:
stvn
2021-04-17 00:44:49 -07:00
parent 33e499ea04
commit a99c95da64
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
openedx/core/djangoapps/discussions/views.py
View File

@@ -120,7 +120,11 @@ class DiscussionsConfigurationView(APIView):
Serialize data into a dictionary, to be used as a response
"""
payload = super().to_representation(instance)
lti_configuration = LtiSerializer(instance.lti_configuration)
lti_configuration_data = {}
supports_lti = instance.supports('lti')
if supports_lti:
lti_configuration = LtiSerializer(instance.lti_configuration)
lti_configuration_data = lti_configuration.data
payload.update({
'features': {
'discussion-page',
@@ -128,7 +132,7 @@ class DiscussionsConfigurationView(APIView):
'lti',
'wcag-2.1',
},
'lti_configuration': lti_configuration.data,
'lti_configuration': lti_configuration_data,
'plugin_configuration': instance.plugin_configuration,
'providers': {
'active': instance.provider_type or DEFAULT_PROVIDER_TYPE,