Skip to content

Conversation

@gumaerc
Copy link
Contributor

@gumaerc gumaerc commented Nov 13, 2025

What are the relevant tickets?

Closes https://github.com/mitodl/hq/issues/9194

Description (What does it do?)

This PR makes some style updates to the CTA buttons in the Learn dashboard, described in the issue above.

Screenshots (if appropriate):

image

How can this be tested?

  • Make sure you have an instance of MITx Online up and running and connected to the same Keycloak / APISIX instance as Learn as described in the README
  • In your instance of MITx Online, make sure you have a B2B organization with a contract and some courses added to said contract, and that your user is part of said organization
  • Visit your org dashboard in Learn and verify that the logic surrounding the CTA text / icon reflects what's described in the issue above:
    • Courses that haven't been started show "Start Course" or "Start Module" with no arrow
    • Courses that have been started say "Continue" and have a right facing arrow
    • Courses that are completed or expired show "View Course" or "View Module" with no arrow

@gumaerc gumaerc added the Needs Review An open Pull Request that is ready for review label Nov 13, 2025
@gumaerc gumaerc changed the title adjust CTA buttons to new width of 124px, set new button text and end… more dashboard CTA adjustments Nov 13, 2025
@abeglova abeglova assigned abeglova and unassigned abeglova Nov 14, 2025
Copy link
Contributor

@ChristopherChudzicki ChristopherChudzicki left a comment

Choose a reason for hiding this comment

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

👍

@gumaerc gumaerc merged commit e9a546c into main Nov 14, 2025
13 checks passed
@gumaerc gumaerc deleted the cg/more-cta-adjustments branch November 14, 2025 15:32
@odlbot odlbot mentioned this pull request Nov 17, 2025
16 tasks
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.

4 participants