Michael Wheeler
Michael Wheeler
The footer on legacy pages was missing the following links specified in `TC_LEARNER_3`: - FAQ - Blog - Donate - Privacy Policy  MFE is missing all links, but I'm...
Formerly `DataPrivacyLaunchRequest`, the message name has been updated to `LtiDataPrivacyLaunchRequest` in [the most recent spec](https://www.imsglobal.org/spec/lti-priv/v1p0/).
The `test-publish` job fails on PRs from dependabot forks (and probably all forks). ``` Notice: Attempting to perform trusted publishing exchange to retrieve a temporary short-lived API token for authentication...
The `test-publish` job will fail with running from a fork. https://github.com/academic-innovation/django-lti/blob/72703a6c375d1ad1e613d91a15a33e7dffef7249/.github/workflows/packaging.yml#L26-L43
Test publishing workflow does not seem to be honoring the `skip-existing` input. See https://github.com/pypa/gh-action-pypi-publish/issues/201.
The public/private key fields in the `RegistrationAdmin` are not frequently used, so they should be added to a [fieldset](https://docs.djangoproject.com/en/4.2/ref/contrib/admin/#django.contrib.admin.ModelAdmin.fieldsets) to reduce confusion.
This value can optionally be provided by the platform, in which case it will be used by the tool when requesting an auth token.
Canvas appears to send the placement from which a tool was launched in the `https://www.instructure.com/placement` claim.
This library could use the [LTI 1.1 migration claim](https://www.imsglobal.org/spec/lti/v1p3/migr#lti-1-1-migration-claim) in a few ways. 1. Add fields for LTI 1.1 IDs to `LtiUser`, `LtiContext`, and `LtiPlatformInstance` 2. Sync those fields when...