Skip to content

Conversation

@jonkafton
Copy link
Contributor

@jonkafton jonkafton commented Jun 17, 2024

What are the relevant tickets?

Displaying price on learning resources #1098

Description (What does it do?)

  • Where courses have the option to be taken free, or paid for with a certificate, display the certificate price in the certificate badge separately for the resource price, "free". Common for MITx courses:
image
  • Some prices are not properly ingested but the free boolean on the resource data is reliable. Where free: false and price is zero, display "Paid". Currently the case with Sloan course on RC:
image
  • Where courses have a range of prices we display e.g. "$950 - $999". If no free option, it is displayed as the course price. This fix #1085 makes the scenario rare/ non existent with current data.
image
  • If the course has a free option, the price range is displayed on the certificate badge.
image
  • Where prices are not to the dollar, displays to 2 decimal places.

How can this be tested?

Run ./manage.py update_index --all for prices update.

Navigate to the search page, /search. The "Certificate" and "Offered By" filters are useful for testing the various scenarios:

  • Free course without certificate option displays "Free"
  • Free course with paid certificate option displays the certificate price and "Free"
  • Free course with paid certificate option range displays the certificate price range and "Free"
  • Paid course without certificate option displays the course price
  • Free course with empty prices array displays "Free"
  • Paid course that has zero price (prices not ingested correctly) displays "Paid"

@jonkafton jonkafton changed the title Display certificate price for free courses Learning resource price display fixes Jun 18, 2024
@jonkafton jonkafton changed the title Learning resource price display fixes Updates to learning resource price display Jun 18, 2024
@jonkafton jonkafton added Needs Review An open Pull Request that is ready for review and removed Work in Progress labels Jun 18, 2024
@gumaerc gumaerc self-assigned this Jun 18, 2024
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.

👍

@jonkafton jonkafton merged commit 5fccaa2 into main Jun 18, 2024
@odlbot odlbot mentioned this pull request Jun 18, 2024
6 tasks
@jonkafton jonkafton removed the Needs Review An open Pull Request that is ready for review label Jun 18, 2024
@rhysyngsun rhysyngsun deleted the jk/1098-price-display branch February 7, 2025 20:31
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