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

fix: Update for new version of demo course #1272

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

timmc-edx
Copy link
Contributor

@timmc-edx timmc-edx commented Feb 7, 2024

In openedx/openedx-demo-course#47 the demo course was replaced. This includes a new course key (new name and org) and the course content being moved a few directories deeper. (Ticket: #1273)

Changes:

  • sed -i 's/course-v1:edX+DemoX+Demo_Course/course-v1:Axim+DemoX+demo_course/g' on all files turned up by grep -rl Demo_Course
  • Updated a few other course info locations turned up with grep -nrI DemoX | grep -x demo_course (all in programs/discovery.py)
  • Updated git clone URL and checkout location in provision-lms.sh to use new repo name (not strictly needed).

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)

In openedx/openedx-demo-course#47 the demo
course was replaced. This includes a new course key (new name and org)
and the course content being moved a few directories deeper.

Changes:

- `sed -i 's/course-v1:edX+DemoX+Demo_Course/course-v1:Axim+DemoX+demo_course/g'`
  on all files turned up by `grep -rl Demo_Course`
- Updated a few other course info locations turned up with
  `grep -nrI DemoX | grep -x demo_course` (all in programs/discovery.py)
- Updated git clone URL and checkout location in provision-lms.sh to
  use new repo name (not strictly needed).
@timmc-edx timmc-edx force-pushed the timmc/fix-demo-course branch from c7c5a74 to c2e6f71 Compare February 7, 2024 18:24
@timmc-edx timmc-edx changed the title fix: Use new location for demo course base XML file fix: Update for new version of demo course Feb 7, 2024
@timmc-edx
Copy link
Contributor Author

timmc-edx commented Feb 9, 2024

Only failing for ecommerce+lms. There are still conflicting demo course references somewhere. I had hoped to fix this by changing mentions of edX as a partner in provision-ecommerce.sh along with changes in openedx-unsupported/ecommerce@master...timmc-edx:ecommerce:timmc/new-demo-course but no luck -- there are now duplicate partner code entries. Maybe requires reverting the SQL changes in this PR and re-running update-dbs-init-sql-scripts.sh with just the right changes to partner codes in devstack and ecommerce?

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.

1 participant