Skip to content

Conversation

@ChristopherChudzicki
Copy link
Contributor

@ChristopherChudzicki ChristopherChudzicki commented Nov 24, 2025

What are the relevant tickets?

Description (What does it do?)

This PR adds the requirements display for program pages in MIT Learn.

Screenshots (if appropriate):

Screenshot 2025-11-24 at 6 53 51 PM Screenshot 2025-11-24 at 6 54 01 PM Screenshot 2025-11-24 at 6 54 18 PM Screenshot 2025-11-24 at 6 55 44 PM

How can this be tested?

  1. Prerequisites:
    • MIT Learn <> MITxOnline integration setup as directed in this repo readme.
    • An MITxOnline program with required and elective courses, published. Also a corresponding CMS page with "include in learn catalog" set to true.
    • ETL set up to ingest local MITxOnline data, e.g.,
      MITX_ONLINE_BASE_URL=http://mitxonline.odl.local:8013/
      MITX_ONLINE_COURSES_API_URL=http://host.docker.internal:8013/api/v2/courses/
      MITX_ONLINE_PROGRAMS_API_URL=http://host.docker.internal:8013/api/v2/programs/
      
  2. Run ./manage.py backpopulate_mitxonline_data
  3. Visit http://open.odl.local:8062/programs/PROGRAM_READABLE_ID.
  4. Check that the required courses show up as course cards; clicking them should open the drawer. Mobile view should show cards as in screenshot above.
  5. Check that the elective courses show up with correct value for "take this many electives" display.
  6. Check that the required + elective course section titles are dynamic, reading from data in http://mitxonline.odl.local:8013/admin/courses/program/PROGRAM_ID/change/

@ChristopherChudzicki ChristopherChudzicki added the Needs Review An open Pull Request that is ready for review label Nov 25, 2025
@gumaerc gumaerc self-assigned this Nov 26, 2025
Copy link
Contributor

@gumaerc gumaerc left a comment

Choose a reason for hiding this comment

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

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review An open Pull Request that is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants