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

Adding new cart designs #2505

Merged
merged 12 commits into from
Jan 14, 2025
Merged

Adding new cart designs #2505

merged 12 commits into from
Jan 14, 2025

Conversation

annagav
Copy link
Contributor

@annagav annagav commented Jan 10, 2025

What are the relevant tickets?

Related to https://github.com/mitodl/hq/issues/5662

Description (What does it do?)

Adding new cart designs

There are some changes that are under the feature flag:

  • everything that is related to the cart

And some that are not:

  • like the certificate availability and price info
  • and the "enroll for free" button

Note: This PR does not include the addition of the new confirmation dialog. This will be added in a separate PR.

Screenshots (if appropriate):

The flag turned off:

Screenshot 2025-01-10 at 7 25 04 AM Screenshot 2025-01-10 at 7 51 16 AM

The flag turned on:

Screenshot 2025-01-10 at 9 05 47 AM Screenshot 2025-01-10 at 9 20 18 AM

How can this be tested?

Switch on and off the feature flag new-cart-design

@annagav annagav changed the title Ag/cart design Adding new cart designs Jan 10, 2025
@annagav annagav force-pushed the ag/cart_design branch 2 times, most recently from ba3b188 to f178025 Compare January 13, 2025 13:27
@annagav annagav merged commit b06be05 into main Jan 14, 2025
7 checks passed
@annagav annagav deleted the ag/cart_design branch January 14, 2025 01:48
@pdpinch
Copy link
Member

pdpinch commented Jan 14, 2025

Let me know how/when I can see this on RC.

@odlbot odlbot mentioned this pull request Jan 14, 2025
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants