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

feat: add a flag to hide course prices in enrollment page when zero #6

Merged

Conversation

tecoholic
Copy link
Member

Description

This commit adds "hide_course_price_when_zero" attribute to the EnterpriseCustomer model which can be enabled to hide the course price information from showing up in the enrollment landing page, when the final price of the course is Zero.

Internal-ref: https://tasks.opencraft.com/browse/BB-7541

This is a nutmeg backport of openedx#1788

tecoholic added 2 commits July 6, 2023 16:05
…zero

This commit adds "hide_course_price_when_zero" attribute to the EnterpriseCustomer
model which can be enabled to hide the course price information from showing up
in the enrollment landing page, when the final price of the course is Zero.

Internal-ref: https://tasks.opencraft.com/browse/BB-7541
Copy link
Member

@Agrendalath Agrendalath left a comment

Choose a reason for hiding this comment

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

@tecoholic, please bump the version before merging.

👍

  • I tested this: checked that the code matches the upstream PR
  • I read through the code
  • I checked for accessibility issues: n/a
  • Includes documentation: n/a
  • I made sure any change in configuration variables is reflected in the corresponding client's configuration-secure repository: n/a

@tecoholic tecoholic merged commit bbafc5e into opencraft-release/nutmeg.2 Jul 10, 2023
@tecoholic tecoholic deleted the tecoholic/BB-7541-nutmeg-backport branch July 10, 2023 13:30
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.

2 participants