- Apr 13, 2021
-
-
Giovanni Cimolin da Silva authored
-
Giovanni Cimolin da Silva authored
This commit improves the functionality of the grade publishing method. Improvements: * Doesn't publish grade to LMS if this isn't a graded block. * Doesn't try to publish grades when LineItem is not linked (`resourceLinkId` is empty) * Improves logging and properly logs exceptions
-
Giovanni Cimolin da Silva authored
This commit re-enables the programmatic grades support on LTI AGS. This allows complex tools to manage grades on their own. The default value keeps the old behavior (declarative mode).
-
- Apr 12, 2021
- Apr 09, 2021
-
-
stvn authored
Fall-back to the old path. Once the LMS has fully switched over to this new path [1], we can remove the legacy (LMS) import support here. - [1] https://github.com/edx/edx-platform/pull/27289
-
- Mar 22, 2021
-
-
Soban Javed authored
-
- Mar 19, 2021
-
-
Ned Batchelder authored
-
- Mar 18, 2021
-
-
Giovanni Cimolin da Silva authored
* Improve content presentation and add multiple content display support This commit: * Improves LTI content presentation handling, and correctly displays LTI DL content in the LMS * Adds support for presenting multiple DL content items on the same block * Addressing review comments * Nits * Improve test and fix issue [BD-24][TNL-8072]
-
- Mar 09, 2021
-
-
edX requirements bot authored
* Python code cleanup by the cleanup-python-code Jenkins job. This pull request was generated by the cleanup-python-code Jenkins job, which ran ``` add_common_constraint; make upgrade ``` The following packages were installed: `edx-repo-tools` * Added --django-settings-module to pylint * Changed position * Remove django-settings-module from pylint command * Add django-not-configured in pylintrc Co-authored-by:
Muhammad Soban Javed <58461728+iamsobanjaved@users.noreply.github.com>
-
- Mar 03, 2021
-
-
Giovanni Cimolin da Silva authored
[BD-24][TNL-7919] LTI Advantage Names and Roles provisioning service implementation direction (#142) * Add wip ADR about NRPS * Nit * Update docs/decisions/0004-lti-advantage-nrps.rst Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com> * Update docs/decisions/0004-lti-advantage-nrps.rst Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com> * Address nits * Write a bit more * Update docs/decisions/0004-lti-advantage-nrps.rst Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com> * Update docs/decisions/0004-lti-advantage-nrps.rst Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com> * Update docs/decisions/0004-lti-advantage-nrps.rst Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com> * Update docs/decisions/0004-lti-advantage-nrps.rst Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com> * Update docs/decisions/0004-lti-advantage-nrps.rst Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com> * Nits Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com>
-
Giovanni Cimolin da Silva authored
* Improve logging and error message presentation * Add full deep linking content data on studio * Internationalize LTI 1.3 templates * Add translation tooling Signed-off-by:
Giovanni Cimolin da Silva <giovannicimolin@gmail.com> * Update translations * Address review comments * Additional logging improvements * Address review comments, fix test Signed-off-by:
Giovanni Cimolin da Silva <giovannicimolin@gmail.com>
-
- Feb 25, 2021
-
-
stvn authored
* Commits: chore: Bump version to 2.7.1 fix: Manually backfill lti_config dicts
-
stvn authored
-
stvn authored
The difficulty here is that the empty string in an invalid JSON dictionary (as opposed to None/NULL or '{}' which is just "blank"); attempts to instantiate models would fail, given the invalid data. This meant we couldn't use the Django ORM to handle the migration entirely; we need to craft some raw SQL to work-around these checks and limitations. Related-To: 42a9e342 Related-To: https://github.com/edx/xblock-lti-consumer/pull/144
-
- Feb 24, 2021
-
-
stvn authored
* Commits: fix: Manually backfill config_id UUIDs again
-
stvn authored
The previous backfills handled NULL entries; this handles empty strings. The difficulty here was that since the empty string in an invalid UUID (as opposed to None/NULL which is just "blank"), attempts to instantiate models would fail, given the invalid data. This meant we couldn't use the Django ORM to handle the migration entirely; we need to craft some raw SQL to work-around these checks and limitations.
-
- Feb 23, 2021
-
-
Giovanni Cimolin da Silva authored
* Implement ltiResourceLink content presentation Signed-off-by:
Giovanni Cimolin da Silva <giovannicimolin@gmail.com> * Add tests * Fix serializer required fields issue Signed-off-by:
Giovanni Cimolin da Silva <giovannicimolin@gmail.com> * Update version and changelog
-
- Feb 16, 2021
-
-
Ned Batchelder authored
-
David Joy authored
-
David Joy authored
fix: Migrate the UUID backfill again
-
Shimul Chowdhury authored
-
Shimul Chowdhury authored
Add test for image Fix broken tests Add `html` content type serializer
-
Shimul Chowdhury authored
Fix broken tests Add serializer for `link` content type
-
Shimul Chowdhury authored
-
- Feb 15, 2021
-
-
Saad Yousaf authored
[TNL-7520] - Handle case when modal height and width and set to empty in LTI component in studio.
-
- Feb 08, 2021
-
-
SaadYousaf authored
-
- Feb 04, 2021
-
-
stvn authored
To correct the, in our case, single database entry that snuck in without a UUID during the schema change. Steps: - Make the field editable - Add `config_id` to all models missing it - Make the field noneditable again
-
- Jan 29, 2021
-
-
Giovanni Cimolin da Silva authored
This implements the endpoint that LTI tools use to return the deep linking configuration to the LMS, along with the `ltiResourceLink` type to allow testing with demo tool. Signed-off-by:
Giovanni Cimolin da Silva <giovannicimolin@gmail.com>
-
Giovanni Cimolin da Silva authored
Signed-off-by:
Giovanni Cimolin da Silva <giovannicimolin@gmail.com>
-
- Jan 25, 2021
-
-
Usama Sadiq authored
Python Code Cleanup
-
usamasadiq authored
-
- Jan 21, 2021
-
-
edX requirements bot authored
-
- Jan 20, 2021
-
-
stvn authored
* Commits: Fix migration `config_id` conflict.
-
Kshitij Sobti authored
A previous PR (#130) added a new unique field with a default value to the LtiConfiguration model. Such cases need special handling in migraiton, however the previous PR did not include that. This commit fixes the migration to include the special handling for new unique fields.
-
- Jan 19, 2021
-
-
Ned Batchelder authored
-
Giovanni Cimolin da Silva authored
Remove the OIDC view from XBlock and return OIDC url directly.
-
- Jan 15, 2021
-
-
Sarina Canelake authored
[BD-03] [TNL-7742] [BB-3370] Add LTI 1.1 config on model
-
Kshitij Sobti authored
-
- Dec 18, 2020
-
-
stvn authored
* Commits: Skip migration when no LTI XBlock location exists
-