* 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>

[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>

* 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>

* Commits:
  chore: Bump version to 2.7.1
  fix: Manually backfill lti_config dicts

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

* Commits:
  fix: Manually backfill config_id UUIDs again

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.

* 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

fix: Migrate the UUID backfill again

Add test for image

Fix broken tests

Add `html` content type serializer

Fix broken tests

Add serializer for `link` content type

[TNL-7520] - Handle case when modal height and width and set to empty in LTI component in studio.

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

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>

Signed-off-by: Giovanni Cimolin da Silva <giovannicimolin@gmail.com>

Python Code Cleanup

* Commits:
  Fix migration `config_id` conflict.

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.

Remove the OIDC view from XBlock and return OIDC url directly.

[BD-03] [TNL-7742] [BB-3370] Add LTI 1.1 config on model

* Commits:
  Skip migration when no LTI XBlock location exists

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).

* bump up the version

* update changelog

[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

* 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