Skip to content
This repository was archived by the owner on Oct 31, 2025. It is now read-only.

feat: add credit seat metadata fields in Publisher UI#1057

Merged
feanil merged 1 commit intoopenedx-unsupported:masterfrom
rjv31:feat/credit-seat-metadata-ui
Oct 21, 2025
Merged

feat: add credit seat metadata fields in Publisher UI#1057
feanil merged 1 commit intoopenedx-unsupported:masterfrom
rjv31:feat/credit-seat-metadata-ui

Conversation

@rjv31
Copy link
Contributor

@rjv31 rjv31 commented Oct 7, 2025

This PR adds UI support in frontend-app-publisher to manage credit seat metadata for course runs — including credit provider, credit hours, and upgrade deadline.

Verified credit fields appear only for credit-type seats and send correct payloads.

Related Ticket - https://2u-internal.atlassian.net/browse/PROD-4432

@rjv31 rjv31 force-pushed the feat/credit-seat-metadata-ui branch 4 times, most recently from 3895582 to 1dd5185 Compare October 8, 2025 11:35
@codecov
Copy link

codecov bot commented Oct 8, 2025

Codecov Report

❌ Patch coverage is 73.52941% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.08%. Comparing base (7ae97c0) to head (72219b1).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
src/components/CreateCourseRunPage/index.jsx 0.00% 9 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1057      +/-   ##
==========================================
+ Coverage   71.08%   72.08%   +0.99%     
==========================================
  Files         133      133              
  Lines        3452     3485      +33     
  Branches      980     1000      +20     
==========================================
+ Hits         2454     2512      +58     
+ Misses        955      933      -22     
+ Partials       43       40       -3     

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

@rjv31 rjv31 force-pushed the feat/credit-seat-metadata-ui branch 7 times, most recently from 0c5cfb6 to 6dc9b91 Compare October 10, 2025 21:31
@rjv31 rjv31 marked this pull request as ready for review October 12, 2025 03:54
@rjv31 rjv31 force-pushed the feat/credit-seat-metadata-ui branch 17 times, most recently from da27e7c to 6e1168d Compare October 19, 2025 08:07
@rjv31 rjv31 force-pushed the feat/credit-seat-metadata-ui branch 4 times, most recently from 138a2f4 to 8953c3b Compare October 19, 2025 09:17
@rjv31 rjv31 force-pushed the feat/credit-seat-metadata-ui branch from 8953c3b to 72219b1 Compare October 19, 2025 10:47
@rjv31 rjv31 requested review from a team and skumargupta83 October 19, 2025 12:41
@rjv31
Copy link
Contributor Author

rjv31 commented Oct 21, 2025

Hey @feanil , @openedx/committers-frontend-app-publisher could anyone please check and help review/merge this PR

@feanil feanil merged commit 9f54c52 into openedx-unsupported:master Oct 21, 2025
6 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants