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

feat: Created API for programs live page zoom lti (#29763)

Browse Source 
* feat: Created API for programs live page zoom lti

* refactor: Merged similar code by inheritance

* refactor: removed duplicates and resolved tight coupling issues

* refactor: Decoupled views
This commit is contained in:
Ahtisham Shahid
2022-01-21 12:26:47 +05:00
committed by GitHub
parent 4e22a38ca5
commit e63fb2e01a
8 changed files with 262 additions and 37 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
openedx/core/djangoapps/discussions/admin.py
View File

@@ -7,8 +7,8 @@ from simple_history.admin import SimpleHistoryAdmin
from openedx.core.djangoapps.config_model_utils.admin import StackedConfigModelAdmin
from .forms import ProgramDiscussionsConfigurationForm
from .models import DiscussionsConfiguration, ProgramDiscussionsConfiguration
from .forms import ProgramDiscussionsConfigurationForm, ProgramLiveConfigurationForm
from .models import DiscussionsConfiguration, ProgramDiscussionsConfiguration, ProgramLiveConfiguration
from .models import ProviderFilter
@@ -112,6 +112,31 @@ class ProviderFilterAdmin(StackedConfigModelAdmin):
)
class ProgramLiveConfigurationAdmin(SimpleHistoryAdmin):
"""
Customize the admin interface for the program live configuration
"""
form = ProgramLiveConfigurationForm
fieldsets = (
(None, {
'fields': ('program_uuid', 'enabled', 'lti_configuration', 'pii_share_username', 'pii_share_email',
'provider_type'),
}),
)
search_fields = (
'program_uuid',
'enabled',
'provider_type',
)
list_filter = (
'enabled',
'provider_type',
)
admin.site.register(DiscussionsConfiguration, DiscussionsConfigurationAdmin)
admin.site.register(ProgramDiscussionsConfiguration, ProgramDiscussionsConfigurationAdmin)
admin.site.register(ProgramLiveConfiguration, ProgramLiveConfigurationAdmin)
admin.site.register(ProviderFilter, ProviderFilterAdmin)

30
openedx/core/djangoapps/discussions/forms.py
View File

@@ -3,7 +3,7 @@ Forms for discussions.
"""
from django import forms
from .models import ProgramDiscussionsConfiguration
from .models import ProgramDiscussionsConfiguration, ProgramLiveConfiguration
class ProgramDiscussionsConfigurationForm(forms.ModelForm):
@@ -32,3 +32,31 @@ class ProgramDiscussionsConfigurationForm(forms.ModelForm):
class Meta:
model = ProgramDiscussionsConfiguration
fields = '__all__'
class ProgramLiveConfigurationForm(forms.ModelForm):
"""
Custom ProgramLiveConfigurationForm form for admin page
"""
pii_share_username = forms.BooleanField(required=False, initial=False)
pii_share_email = forms.BooleanField(required=False, initial=False)
def __init__(self, *args, **kwargs):
super().__init__(*args, **kwargs)
if self.instance.lti_configuration:
self.fields['pii_share_username'].initial = self.instance.lti_configuration.pii_share_username
self.fields['pii_share_email'].initial = self.instance.lti_configuration.pii_share_email
def save(self, commit=True):
pii_share_username = self.cleaned_data.get('pii_share_username', False)
pii_share_email = self.cleaned_data.get('pii_share_email', False)
lti_configuration = self.cleaned_data.get('lti_configuration', None)
if lti_configuration:
lti_configuration.pii_share_username = pii_share_username
lti_configuration.pii_share_email = pii_share_email
lti_configuration.save()
return super().save(commit=commit)
class Meta:
model = ProgramLiveConfiguration
fields = '__all__'

76
openedx/core/djangoapps/discussions/migrations/0008_auto_20220119_0746.py Normal file
View File

@@ -0,0 +1,76 @@
# Generated by Django 3.2.11 on 2022-01-19 07:46
from django.conf import settings
from django.db import migrations, models
import django.db.models.deletion
import django.utils.timezone
import model_utils.fields
import simple_history.models
class Migration(migrations.Migration):
dependencies = [
('lti_consumer', '0013_auto_20210712_1352'),
migrations.swappable_dependency(settings.AUTH_USER_MODEL),
('discussions', '0007_add_discussion_topic_link'),
]
operations = [
migrations.AlterField(
model_name='historicalprogramdiscussionsconfiguration',
name='enabled',
field=models.BooleanField(default=True, help_text='If disabled, the LTI in the associated program will be disabled.'),
),
migrations.AlterField(
model_name='historicalprogramdiscussionsconfiguration',
name='provider_type',
field=models.CharField(help_text="The LTI provider's id", max_length=50, verbose_name='LTI provider'),
),
migrations.AlterField(
model_name='programdiscussionsconfiguration',
name='enabled',
field=models.BooleanField(default=True, help_text='If disabled, the LTI in the associated program will be disabled.'),
),
migrations.AlterField(
model_name='programdiscussionsconfiguration',
name='provider_type',
field=models.CharField(help_text="The LTI provider's id", max_length=50, verbose_name='LTI provider'),
),
migrations.CreateModel(
name='ProgramLiveConfiguration',
fields=[
('created', model_utils.fields.AutoCreatedField(default=django.utils.timezone.now, editable=False, verbose_name='created')),
('modified', model_utils.fields.AutoLastModifiedField(default=django.utils.timezone.now, editable=False, verbose_name='modified')),
('program_uuid', models.CharField(db_index=True, max_length=50, primary_key=True, serialize=False, verbose_name='Program UUID')),
('enabled', models.BooleanField(default=True, help_text='If disabled, the LTI in the associated program will be disabled.')),
('provider_type', models.CharField(help_text="The LTI provider's id", max_length=50, verbose_name='LTI provider')),
('lti_configuration', models.ForeignKey(blank=True, help_text='The LTI configuration data for this program/provider.', null=True, on_delete=django.db.models.deletion.SET_NULL, to='lti_consumer.lticonfiguration')),
],
options={
'abstract': False,
},
),
migrations.CreateModel(
name='HistoricalProgramLiveConfiguration',
fields=[
('created', model_utils.fields.AutoCreatedField(default=django.utils.timezone.now, editable=False, verbose_name='created')),
('modified', model_utils.fields.AutoLastModifiedField(default=django.utils.timezone.now, editable=False, verbose_name='modified')),
('program_uuid', models.CharField(db_index=True, max_length=50, verbose_name='Program UUID')),
('enabled', models.BooleanField(default=True, help_text='If disabled, the LTI in the associated program will be disabled.')),
('provider_type', models.CharField(help_text="The LTI provider's id", max_length=50, verbose_name='LTI provider')),
('history_id', models.AutoField(primary_key=True, serialize=False)),
('history_date', models.DateTimeField()),
('history_change_reason', models.CharField(max_length=100, null=True)),
('history_type', models.CharField(choices=[('+', 'Created'), ('~', 'Changed'), ('-', 'Deleted')], max_length=1)),
('history_user', models.ForeignKey(null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='+', to=settings.AUTH_USER_MODEL)),
('lti_configuration', models.ForeignKey(blank=True, db_constraint=False, help_text='The LTI configuration data for this program/provider.', null=True, on_delete=django.db.models.deletion.DO_NOTHING, related_name='+', to='lti_consumer.lticonfiguration')),
],
options={
'verbose_name': 'historical program live configuration',
'ordering': ('-history_date', '-history_id'),
'get_latest_by': 'history_date',
},
bases=(simple_history.models.HistoricalChanges, models.Model),
),
]

23
openedx/core/djangoapps/discussions/models.py
View File

@@ -518,10 +518,12 @@ class DiscussionsConfiguration(TimeStampedModel):
)
class ProgramDiscussionsConfiguration(TimeStampedModel):
class AbstractProgramLTIConfiguration(TimeStampedModel):
"""
Associates a program with a discussion provider and configuration
Associates a program with a LTI provider and configuration
"""
class Meta:
abstract = True
program_uuid = models.CharField(
primary_key=True,
@@ -531,7 +533,7 @@ class ProgramDiscussionsConfiguration(TimeStampedModel):
)
enabled = models.BooleanField(
default=True,
help_text=_("If disabled, the discussions in the associated program will be disabled.")
help_text=_("If disabled, the LTI in the associated program will be disabled.")
)
lti_configuration = models.ForeignKey(
LtiConfiguration,
@@ -543,10 +545,9 @@ class ProgramDiscussionsConfiguration(TimeStampedModel):
provider_type = models.CharField(
blank=False,
max_length=50,
verbose_name=_("Discussion provider"),
help_text=_("The discussion provider's id"),
verbose_name=_("LTI provider"),
help_text=_("The LTI provider's id"),
)
history = HistoricalRecords()
def __str__(self):
return f"Configuration(uuid='{self.program_uuid}', provider='{self.provider_type}', enabled={self.enabled})"
@@ -556,7 +557,7 @@ class ProgramDiscussionsConfiguration(TimeStampedModel):
"""
Lookup a program discussion configuration by program uuid.
"""
return ProgramDiscussionsConfiguration.objects.filter(
return cls.objects.filter(
program_uuid=program_uuid
).first()
@@ -614,3 +615,11 @@ class DiscussionTopicLink(models.Model):
f'enabled_in_context={self.enabled_in_context}'
f')'
)
class ProgramLiveConfiguration(AbstractProgramLTIConfiguration):
history = HistoricalRecords()
class ProgramDiscussionsConfiguration(AbstractProgramLTIConfiguration):
history = HistoricalRecords()