Skip to content
Snippets Groups Projects
  1. Sep 08, 2021
  2. Jul 01, 2021
    • Kshitij Sobti's avatar
      refactor: Rename CourseEditLTIFieldsEnabledFlag to... · 6fb86793
      Kshitij Sobti authored
      refactor: Rename CourseEditLTIFieldsEnabledFlag to CourseAllowPIISharingInLTIFlag and use it for LTI1.3
      This commit renames the CourseEditLTIFieldsEnabledFlag to CourseAllowPIISharingInLTIFlag since the aim is to expand its scope to all LTI-related PII sharing. It also removes the current LTI1.3 waffle flag for PII sharing.
      Unverified
      6fb86793
  3. Jun 14, 2021
  4. Apr 13, 2021
  5. Mar 22, 2021
  6. Feb 25, 2021
    • stvn's avatar
      fix: Manually backfill lti_config dicts · 7082a991
      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
      7082a991
  7. Feb 24, 2021
    • stvn's avatar
      fix: Manually backfill config_id UUIDs again · 42a9e342
      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.
      42a9e342
  8. Feb 04, 2021
    • stvn's avatar
      fix: Migrate the UUID backfill again · 8b17e4a6
      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
      8b17e4a6
  9. Jan 29, 2021
  10. Jan 20, 2021
    • Kshitij Sobti's avatar
      Fix migration `config_id` conflict. · 59b7c431
      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.
      Unverified
      59b7c431
  11. Jan 15, 2021
  12. Dec 17, 2020
    • stvn's avatar
      Skip migration when no LTI XBlock location exists · 9f326ee8
      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).
      9f326ee8
  13. Nov 12, 2020
  14. Oct 23, 2020
    • Patrick Cockwell's avatar
      [BD-24] Implement LTI AGS Score Publish Service and Results (#108) · 5fc16b38
      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
      Unverified
      5fc16b38
  15. Oct 05, 2020
  16. Aug 26, 2020
Loading