Skip to content

Conversation

@mbertrand
Copy link
Member

@mbertrand mbertrand commented Aug 27, 2024

What are the relevant tickets?

Backend part of https://github.com/mitodl/hq/issues/5307

Description (What does it do?)

-Run docker compose up or docker compose run --rm web ./manage.py migrate. Migrations should succeed without errors.

  • Check that any OCW courses in your database have runs with a delivery value equal to their parent course (usually ["online", "offline"]. Ditto non-ocw courses (usually ["online"]).
  • Check that any course/program resources in your database have runs with a availability value equal to their parent course.
  • Run the following , they should all still work (You'll need to populate all the relevant env settings first):
    ./manage.py backpopulate_micromasters_data
    ./manage.py backpopulate_mitxonline_data
    ./manage.py backpopulate_mit_edx_data
    ./manage.py backpopulate_sloan_data
    ./manage.py backpopulate_xpro_data
    ./manage.py backpopulate_prolearn_data
    ./manage.py backpopulate_oll_data
    ./manage.py backpopulate_ocw_data --course-name 21w-758-genre-fiction-workshop-fantasy-fall-2016 --skip-contentfiles
    

Afterward, search for http://open.odl.local:8062/search?q=%22Accelerating+Digital+Transformation+with+Algorithmic+Business+Thinking%22 - open the drawer for that course and inspect the api result, the "delivery" attributes should have some differing values by run.

@mbertrand mbertrand force-pushed the mb/run_delivery_availability branch 2 times, most recently from 5566e35 to 8ff7432 Compare September 11, 2024 20:26
@mbertrand mbertrand removed the Blocked label Sep 11, 2024
@mbertrand mbertrand force-pushed the mb/run_delivery_availability branch 4 times, most recently from cf78c46 to 9f9faea Compare September 13, 2024 15:43
@mbertrand mbertrand added Needs Review An open Pull Request that is ready for review project:open-edx and removed Work in Progress project:open-edx labels Sep 13, 2024
@rhysyngsun rhysyngsun self-assigned this Sep 13, 2024
Copy link
Contributor

@rhysyngsun rhysyngsun left a comment

Choose a reason for hiding this comment

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

One small piece of feedback but otherwise is looking good.

@mbertrand mbertrand force-pushed the mb/run_delivery_availability branch 2 times, most recently from a425938 to 147bf39 Compare September 16, 2024 14:12
Copy link
Contributor

@rhysyngsun rhysyngsun left a comment

Choose a reason for hiding this comment

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

LGTM

@rhysyngsun rhysyngsun added Waiting on author and removed Needs Review An open Pull Request that is ready for review labels Sep 16, 2024
@mbertrand mbertrand force-pushed the mb/run_delivery_availability branch from a524a32 to 8359a68 Compare September 17, 2024 15:25
@mbertrand mbertrand merged commit 80267e7 into main Sep 17, 2024
This was referenced Sep 18, 2024
@mbertrand mbertrand deleted the mb/run_delivery_availability branch October 23, 2024 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants