Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: Add a note about the second DB that needs to be migrated. #34937

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

feanil
Copy link
Contributor

@feanil feanil commented Jun 6, 2024

CSMH lives in a second database named student_moudule_history and so
you need to run migrations on that to create the csmh_extended table.

CSMH lives in a second database named `student_moudule_history` and so
you need to run migrations on that to create the csmh_extended table.
@feanil feanil requested a review from kdmccormick June 6, 2024 19:37
Copy link
Member

@kdmccormick kdmccormick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is another example where edx-platform defaults are designed for edX.org-scale sites but then Tutor gets rid of them to simplify small-scale sites and I honestly don't know what the right thing is for the "average" Open edX site. Something to think about when we do the big config cleanup.

Regardless, since CSMH is the default edx-platform configuration, I agree that we should document it here. Thanks.

@feanil feanil merged commit c8aabe4 into master Jun 18, 2024
45 checks passed
@feanil feanil deleted the feanil/update_docs branch June 18, 2024 15:22
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

1 similar comment
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

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.

3 participants