Skip to content

Conversation

@rjv31
Copy link
Contributor

@rjv31 rjv31 commented Sep 22, 2025

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

This PR enhances the existing Discovery API endpoint to support credit seat metadata as part of the eCommerce deprecation effort. Specifically, we now allow credit course runs to include:

  • Credit Provider (e.g., ASU, CO)
  • Credit Hours
  • Upgrade Deadline Override

Changes Introduced:

  • Updated CourseRun seat creation logic (update_or_create_seats, update_or_create_seat_helper) to accept and persist credit metadata.
  • Extended serializers to expose credit_provider, credit_hours, and upgrade_deadline_override fields for credit seats.
  • Added validation to ensure data integrity.

@rjv31 rjv31 force-pushed the PROD-4431/support_credit_seat_metadata branch 2 times, most recently from 36fa03d to ad635b2 Compare September 24, 2025 07:24
@rjv31 rjv31 force-pushed the PROD-4431/support_credit_seat_metadata branch 7 times, most recently from 751926d to 83702c2 Compare September 29, 2025 11:43
@rjv31 rjv31 marked this pull request as ready for review September 29, 2025 12:02
@ankit-sonata
Copy link

@rjv31 I noticed the build is failing due to a failing test case, could you take a look at the test logs and fix the issue so that the build passes?

@rjv31 rjv31 force-pushed the PROD-4431/support_credit_seat_metadata branch from 6a7066a to 8ce6ae4 Compare October 6, 2025 11:00
@rjv31 rjv31 force-pushed the PROD-4431/support_credit_seat_metadata branch from 70d6a26 to e24f45e Compare October 11, 2025 16:38
@rjv31 rjv31 force-pushed the PROD-4431/support_credit_seat_metadata branch from dc9c2d3 to e24f45e Compare November 26, 2025 17:31
@rjv31 rjv31 force-pushed the PROD-4431/support_credit_seat_metadata branch from 78ac0af to cb48e9b Compare December 1, 2025 07:48
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