fix: upgrade lti-consumer-xblock library to install PII sharing dialog fixes

This commit upgrades the version of the lti-consumer-xblock library from version 7.2.2 to version 7.2.3. This new version contains a fix to the LTI PII sharing consent dialog.

Please see the CHANGELOG entry for this version for a full description of the fixes: https://github.com/openedx/xblock-lti-consumer/blob/master/CHANGELOG.rst#723---2023-01-24. The commit message is included below for convenience.

This commit fixes a bug in the PII sharing consent dialog.

The bug resulted in bizarre behavior when there were more than one LTI component in a unit. For example, if there were two LTI inline launches in a unit, two "OK" button would appear in a single component, instead of in their respective components. Another example is that clicking the "View resource in a [modal|new] window" buttons under two LTI components resulted in the "OK" and "Cancel" buttons as well as the PII sharing prompt appearing in a single component, instead of in their respective components.

This is because the dialog-container div that is dynamically created in the Javascript was not scoped to the LTI component, so there was a div with a id of "dialog-container" for each component configured to share PII. When dynamically inserting and removing buttons and the PII sharing prompt, the Javascript would simply find the first div with the dialog-container ID and operate on it, instead of the div appropriate to the component the user is interacting with.
This commit is contained in:
michaelroytman
2023-01-24 16:14:58 -05:00
parent 0139610e11
commit 1cbd5e65d9
3 changed files with 3 additions and 3 deletions

View File

@@ -665,7 +665,7 @@ libsass==0.10.0
# ora2
loremipsum==1.0.5
# via ora2
lti-consumer-xblock==7.2.2
lti-consumer-xblock==7.2.3
# via -r requirements/edx/base.in
lxml==4.9.2
# via

View File

@@ -891,7 +891,7 @@ loremipsum==1.0.5
# via
# -r requirements/edx/testing.txt
# ora2
lti-consumer-xblock==7.2.2
lti-consumer-xblock==7.2.3
# via -r requirements/edx/testing.txt
lxml==4.9.2
# via

View File

@@ -849,7 +849,7 @@ loremipsum==1.0.5
# via
# -r requirements/edx/base.txt
# ora2
lti-consumer-xblock==7.2.2
lti-consumer-xblock==7.2.3
# via -r requirements/edx/base.txt
lxml==4.9.2
# via