Skip to content

Conversation

@NoyanAziz
Copy link
Contributor

@NoyanAziz NoyanAziz commented Jun 27, 2025

This PR aims to introduce an override for creditPurchaseUrl based on an env variable CREDIT_PURCHASE_URL

@codecov
Copy link

codecov bot commented Jun 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.60%. Comparing base (05a0810) to head (27e7020).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #675   +/-   ##
=======================================
  Coverage   97.60%   97.60%           
=======================================
  Files         152      152           
  Lines        1334     1337    +3     
  Branches      229      233    +4     
=======================================
+ Hits         1302     1305    +3     
  Misses         31       31           
  Partials        1        1           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@deborahgu deborahgu left a comment

Choose a reason for hiding this comment

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

looks good. I'd like to see a test on the new logic, but if you want to hold off on the test until monday that's fine with me, because we see the fallback is still there.

@deborahgu deborahgu merged commit 90aa652 into openedx:master Jun 30, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants