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

Zoom lti pro configurations ADR (#29694)

Browse Source 
* feat: Zoom lti pro configurations ADR
This commit is contained in:
Ahtisham Shahid
2022-01-12 10:55:40 +05:00
committed by GitHub
parent 81a9ecb9e7
commit f517cb571f
1 changed files with 42 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

42
lms/djangoapps/learner_dashboard/001-zoom-lti-pro-configuration.rst Normal file
View File

@@ -0,0 +1,42 @@
Configure Zoom Lti in Programs
--------------
Status
======
Approved
Context
=======
The zoom lti pro is needed to be configured for programs (both masters and
regular) to provide video call feature. zoom-lti-pro can be installed by any
user for free to get credentials which can be used to configure as LTI.
But there is no specified model that can be used to map zoom LTI credentials
with programs.
Decisions
=========
We can use the ``program_discussion_configuration`` model to add zoom lti pro
configurations for the program. A new type will be added so we can identify the
type of configuration if it is a discussion or live configuration.
We are going ahead with LTI 1.1 to configure zoom for now due to some
constraints in current implementation.
Consequences
============
It would become possible to add zoom lti configurations in programs with
minimal effort.
Alternatives
============
1. New model ``program_live_configuration`` can be created which will allow us
to map LTI credentials to programs.
2. ``program_discussion_configuration`` model can be renamed to a more generic
name like ``program_lti_configuration`` but it will require major
refactoring of code.