Skip to content

feat: convert AlertModal to TypeScript - #4142

Draft
brian-smith-tcril wants to merge 3 commits into
openedx:release-23.xfrom
brian-smith-tcril:alertmodal-to-ts
Draft

feat: convert AlertModal to TypeScript#4142
brian-smith-tcril wants to merge 3 commits into
openedx:release-23.xfrom
brian-smith-tcril:alertmodal-to-ts

test(AlertModal): add tests for default prop branches

dea6dda
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Feb 25, 2026 in 0s

94.42% (+0.03%) compared to dfc69e2

View this Pull Request on Codecov

94.42% (+0.03%) compared to dfc69e2

Details

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.42%. Comparing base (dfc69e2) to head (dea6dda).
⚠️ Report is 7 commits behind head on release-23.x.

Additional details and impacted files
@@               Coverage Diff                @@
##           release-23.x    #4142      +/-   ##
================================================
+ Coverage         94.39%   94.42%   +0.03%     
================================================
  Files               242      242              
  Lines              4296     4303       +7     
  Branches           1020     1025       +5     
================================================
+ Hits               4055     4063       +8     
+ Misses              237      232       -5     
- Partials              4        8       +4     

☔ 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.