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

fix: Demo course provision script #121

Open
wants to merge 1 commit into
base: pearson-release/olive.stage
Choose a base branch
from

Conversation

kuipumu
Copy link

@kuipumu kuipumu commented Apr 3, 2024

Description

This PR adds a fix for the script used for demo course provision, this script is executed when devstack is provisioned, since the script is broken the devstack task that is responsible to execute this script fails.

Type of Change

  • Modify scripts/provision-demo-course.sh

@Squirrel18
Copy link

This script has been removed in a recent PR: openedx#31811 so I think it is best to add this error and the appropriate solution to our environment setup instructions and close this PR based on the future removal of the script. What do you think, @kuipumu ?

@kuipumu
Copy link
Author

kuipumu commented Apr 8, 2024

@Squirrel18 I think this is also a good option, where do we add the environment setup instructions?

@Squirrel18
Copy link

I send you the link internally. @kuipumu

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