The code here introduces separate tutor ... do init steps for running Django migrations for the sample plugin. This shouldn't be necessary since the plugin is installed with the rest of the Django dependencies which get picked up and migrated automatically.
Not critical, but maybe confusing, and definitely slows init down by 30+ seconds to spin up the container and run the whole no-op migration path.
|
# --------------------------------------------------------------------------- |
|
# Migrations: Run openedx_plugin_sample migrations on init |
|
# --------------------------------------------------------------------------- |
|
|
|
hooks.Filters.CLI_DO_INIT_TASKS.add_item(( |
|
"lms", |
|
"./manage.py lms migrate openedx_plugin_sample", |
|
)) |
|
hooks.Filters.CLI_DO_INIT_TASKS.add_item(( |
|
"cms", |
|
"./manage.py cms migrate openedx_plugin_sample", |
|
)) |
The code here introduces separate
tutor ... do initsteps for running Django migrations for the sample plugin. This shouldn't be necessary since the plugin is installed with the rest of the Django dependencies which get picked up and migrated automatically.Not critical, but maybe confusing, and definitely slows init down by 30+ seconds to spin up the container and run the whole no-op migration path.
sample-plugin/tutor-contrib-sample/tutorsample/plugin.py
Lines 41 to 52 in ccc9a1d