Skip to content
This repository was archived by the owner on Aug 1, 2024. It is now read-only.

feat: do not include demo course when updating devstack initial sql scripts#1024

Merged
rgraber merged 9 commits intomasterfrom
rsgraber/20230221-update-init-sql-process
Feb 22, 2023
Merged

feat: do not include demo course when updating devstack initial sql scripts#1024
rgraber merged 9 commits intomasterfrom
rsgraber/20230221-update-init-sql-process

Conversation

@rgraber
Copy link
Copy Markdown
Contributor

@rgraber rgraber commented Feb 22, 2023

Issue: edx/edx-arch-experiments#152

It was decided that we should just remove the imported demo course from the initial sql scripts because, with the SplitModuleStore, the sql was tied to mongo data. This caused doc was None when attempting to retrieve structure for item with key 639bc9c9dc264f63954f17d3 errors when trying to import the course again during provisioning.


I've completed each of the following or determined they are not applicable:

  • Made a plan to communicate any major developer interface changes (or N/A)

@rgraber rgraber mentioned this pull request Feb 22, 2023
1 task
@rgraber rgraber marked this pull request as ready for review February 22, 2023 13:47
@rgraber rgraber merged commit 1cdbcea into master Feb 22, 2023
@rgraber rgraber deleted the rsgraber/20230221-update-init-sql-process branch February 22, 2023 16:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants