- 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
-
- Dec 17, 2020
-
-
stvn authored
otherwise this blows up, trying to migrate a record with no location (the modulestore lookup throws). Presumably this isn't (wasn't?) an issue in prod already, right? But I did hit this in my devstack, having previously created an LtiConfiguration record via Django Admin (without a location).
-
- Dec 02, 2020
-
-
Ned Batchelder authored
-
Ned Batchelder authored
-
- Dec 01, 2020
-
-
Shimul Chowdhury authored
* bump up the version * update changelog
-
- Nov 20, 2020
-
-
Ned Batchelder authored
-
Ned Batchelder authored
-
Shimul Chowdhury authored
[BD-24] [TNL-7661] [BB-3172] LTI Improvements - Use declarative grading model on XBlock launch (#116) * create default LineItem, WIP grade save * add score to django admin * WIP: find user and save grade to xblock * boolean pragramatic grade interaction flag and optional params in enable_ags method * Submit grades using grade signals * lineitem urls should be optional * lineitem is now readonly in declarative method * test grade_submit called properly * quality issue * raise LTIError * moved listener to signal.py, refactored models.py, added due and start date, updated tests. * use load_block_as_anonymous_user and remove load_block, refactor tests * refactor test to fix quality issue * make lineitems_url required * refactor tests, accept_grades_past_due on check * test accept_grades_past_due * add comma to last items * refactor get_lti_ags_lineitems_url * make sure crum returns user and not None * nitpicks & use maximum score when given score is larger than maximum * fix docstring of load_block_as_anonymous_user
-
- Nov 12, 2020
-
-
Giovanni Cimolin da Silva authored
* Move LTI 1.3 Key management to model This: - Removes the need to load the modulestore on every public keyset endpoint call. - Simplifies the block structure and parent method overrides. - Removes private key, client id and related parameters from XBlock fields It also includes a migration from the data stored in the block to the model. * Cleanup unused test helpers * Version bump * Addressing review comments
-
- Nov 10, 2020
-
-
Giovanni Cimolin da Silva authored
-
- Nov 05, 2020
-
-
Giovanni Cimolin da Silva authored
* Write score linking ADR * Formatting fixes * Nit * Addressing review comments * Final implementation * Nit * Updating ADR given discussions
-
- Oct 23, 2020
-
-
Patrick Cockwell authored
* BD-24 Implement LTI AGS Score Publish Service and Results Service * Address PR comments and add more validation * Address PR comments * Add tests; Fix error with scoreMaximum; Fix quality issues; Adjust user_id results url slightly * Add permissions tests and address other PR comments * Fix quality test * Address PR comments
-
Giovanni Cimolin da Silva authored
* Add ADR for LTI 1.3 variable definition * Update docs/decisions/0002-lti-1p3-variables.rst Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com> * Update docs/decisions/0002-lti-1p3-variables.rst Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com> * Update docs/decisions/0002-lti-1p3-variables.rst Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com> * Update docs/decisions/0002-lti-1p3-variables.rst Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com> * Update docs/decisions/0002-lti-1p3-variables.rst Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com> * Update docs/decisions/0002-lti-1p3-variables.rst Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com> * Update docs/decisions/0002-lti-1p3-variables.rst Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com> * nit * Remove student role from instructor/staff launches Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com>
-
- Oct 13, 2020
-
-
Nils Diefenbach authored
As stated by @giovannicimolin: Due to a recent change in `Chrome`_ and `Firefox`_ and the way they handle cookies, it’s not possible to access cookies marked as ``SameSite=None`` if they are not also Secure. This is to help avoid a few CSRF attacks. Secure cookies are only available when the request is done through `HTTPS`_, which is not the case on the devstack. The workaround for this behaviour is to set ``DCS_SESSION_COOKIE_SAMESITE`` from ``'Lax'`` to ``None`` in the studio container's ``devstack.py``, when using tools such as ``ngrok``. This commit adds a note about this to the ``README.rst`` of the repository. .. _Chrome:: https://twitter.com/ChromiumDev/status/1293236234932846596 .. _Firefox:: https://hacks.mozilla.org/2020/08/changes-to-samesite-cookie-behavior/ .. See ``Secure`` Attribute as a reference .. _HTTPS:: https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Set-Cookie
-
- Oct 05, 2020
-
-
Giovanni Cimolin da Silva authored
* Add initial version of LineItem Implementation Signed-off-by:
Giovanni Cimolin da Silva <giovannicimolin@gmail.com> * Adding tests * Add LTI tests and fix quality isses * Quality nit * Add missing requirement, upgrade * Squash migrations * Fix merge conflicts * Addressing review comments
-
- Sep 29, 2020
-
-
Giovanni Cimolin da Silva authored
* Add django url testing support * Add tests to django views, implement compat layer * Addressing review comments * Improve tests and test descriptions * Nit
-
- Sep 24, 2020
-
-
Ned Batchelder authored
-
- Sep 22, 2020
-
-
Giovanni Cimolin da Silva authored
* Implement extensions and view support for LTI * Add missing iss to token * Add missing requirement, upgrade requirements * Improving comment * Add missing exception statement * Update lti_consumer/lti_1p3/tests/extensions/rest_framework/test_authentication.py Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com> Co-authored-by:
Ned Batchelder <ned@nedbatchelder.com>
-
- Sep 15, 2020
-
-
Ned Batchelder authored
Fix the installation
-