Skip to content

Commit

Permalink
docs: Add note related to CCX compatibility
Browse files Browse the repository at this point in the history
  • Loading branch information
kuipumu committed Apr 24, 2023
1 parent 2bad7fe commit 4c7aa09
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions docs/decisions/0008-lti-1p3-puggable-lti-configuration.rst
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,14 @@ Note: We aren't completely clear on what might need to change appart from
_get_lti_1p3_consumer and get_lti_1p3_launch_info in xblock-lti-consumer
to allow the use of LTI 1.3 external configurations.

We also need to modify the LtiConfiguration model to sync the changes on
the config store field and external ID field on all CCX XBlocks related to
the master courser XBlock beign modified, this could be achieved by two method,
first by copying the config store and external ID values when a CCX XBlock
LtiConfiguration is beign created and by also sync changes from the master
LtiConfiguration with a post save signal, in that signal we can iterate over
any CCX LtiConfiguration related and sync the config ID and external ID values.

Changes on lti_store
~~~~~~~~~~~~~~~~~~~~

Expand Down

0 comments on commit 4c7aa09

Please sign in to comment.