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

display course number on program enroll modal #2048

Merged
merged 1 commit into from
Jan 5, 2024

Conversation

rachellougee
Copy link
Contributor

What are the relevant tickets?

https://github.com/mitodl/hq/issues/2623#issuecomment-1875972496

Description (What does it do?)

display the course number instead of course ID on program enroll modal

Screenshots (if appropriate):

image

How can this be tested?

Go to program enrollment page (It requires setting up courses, programs locally)
Click on "Enroll now"
A upgrade enrollment dialog should popup, it should list all the courses in that program.
Note that the display name should be course title + course number
The screen reader should read the course title and course number as well

Copy link
Contributor

@JenniWhitman JenniWhitman left a comment

Choose a reason for hiding this comment

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

ezpz looks good.

@rachellougee rachellougee merged commit cb7d78f into main Jan 5, 2024
3 checks passed
@rachellougee rachellougee deleted the update-course-list-on-program-page branch January 5, 2024 14:34
@odlbot odlbot mentioned this pull request Jan 8, 2024
2 tasks
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.

2 participants