Skip to content

Conversation

@feanil
Copy link
Contributor

@feanil feanil commented Sep 4, 2025

This is based on the https://github.com/overhangio/cookiecutter-tutor-plugin

The major changes are:

  • We moved the test workflow file to the top level .github repo.
  • The instructions in the README are updated to match the location of
    this plugin in the openedx-tutor-plugins repo.
  • We added the enable_lti_store.sh init task to run migrations and flip
    a waffle flag.
  • We added the openedx-dockerfile-post-python-requirements patch to
    install the openedx-ltistore library.
  • We removed a bunch of boilerplate from the plugin.py file that was not
    relevant.

@feanil feanil force-pushed the feanil/lti_store_plugin branch 4 times, most recently from 2923a33 to 6aef75f Compare September 4, 2025 20:10
This is based on the https://github.com/overhangio/cookiecutter-tutor-plugin

The major changes are:
* We moved the test workflow file to the top level `.github` repo.
* The instructions in the README are updated to match the location of
  this plugin in the openedx-tutor-plugins repo.
* We added the enable_lti_store.sh init task to run migrations and flip
  a waffle flag.
* We added the openedx-dockerfile-post-python-requirements patch to
  install the openedx-ltistore library.
* We removed a bunch of boilerplate from the plugin.py file that was not
  relevant.
@feanil feanil force-pushed the feanil/lti_store_plugin branch from 6aef75f to dddacb5 Compare September 5, 2025 13:17
@feanil feanil requested a review from MaferMazu September 5, 2025 14:41
There are issues with the base ltistore code so have the plugin install
a branch off of my fork for now where we can iterate and debug while we
test it out with the latest versions of edx-platform.
@feanil feanil merged commit ec3862f into main Sep 8, 2025
3 checks passed
@feanil feanil deleted the feanil/lti_store_plugin branch September 8, 2025 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants