richie-education 3.1.3-dev11 → 3.1.3-dev15

package/js/api/joanie.ts

@@ -127,8 +127,8 @@ export const getRoutes = () => {
     },
     organizations: {
       get: `${baseUrl}/organizations/:id/`,
-      courseProductRelations: {
-        get: `${baseUrl}/organizations/:organization_id/course-product-relations/:id/`,
+      offers: {
+        get: `${baseUrl}/organizations/:organization_id/offers/:id/`,
       },
       courses: {
         get: `${baseUrl}/organizations/:organization_id/courses/:id/`,
@@ -156,8 +156,8 @@ export const getRoutes = () => {
     courseRuns: {
       get: `${baseUrl}/course-runs/:id/`,
     },
-    courseProductRelations: {
-      get: `${baseUrl}/course-product-relations/:id/`,
+    offers: {
+      get: `${baseUrl}/offers/:id/`,
     },
     contractDefinitions: {
       previewTemplate: `${baseUrl}/contract_definitions/:id/preview_template/`,
@@ -470,12 +470,12 @@ const API = (): Joanie.API => {
         ).then(checkStatus);
       },
     },
-    courseProductRelations: {
-      get: (filters?: Joanie.CourseProductRelationQueryFilters) => {
+    offers: {
+      get: (filters?: Joanie.OfferQueryFilters) => {
         return fetchWithJWT(
           filters?.organization_id
-            ? buildApiUrl(ROUTES.organizations.courseProductRelations.get, filters)
-            : buildApiUrl(ROUTES.courseProductRelations.get, filters),
+            ? buildApiUrl(ROUTES.organizations.offers.get, filters)
+            : buildApiUrl(ROUTES.offers.get, filters),
         ).then(checkStatus);
       },
     },

package/js/components/ContractFrame/OrganizationContractFrame.spec.tsx

@@ -6,14 +6,10 @@ import { IntlProvider } from 'react-intl';
 import fetchMock from 'fetch-mock';
 import { QueryStateFactory } from 'utils/test/factories/reactQuery';
 import { RichieContextFactory as mockRichieContextFactory } from 'utils/test/factories/richie';
-import {
-  ContractFactory,
-  CourseProductRelationFactory,
-  OrganizationFactory,
-} from 'utils/test/factories/joanie';
+import { ContractFactory, OfferFactory, OrganizationFactory } from 'utils/test/factories/joanie';
 import { createTestQueryClient } from 'utils/test/createTestQueryClient';
 import JoanieSessionProvider from 'contexts/SessionContext/JoanieSessionProvider';
-import { isCourseProductRelation } from 'types/Joanie';
+import { isOffer } from 'types/Joanie';
 import { Props } from './AbstractContractFrame';
 import { OrganizationContractFrame } from '.';
 
@@ -77,31 +73,28 @@ describe('OrganizationContractFrame', () => {
 
   it.each([
     {
-      label: 'contractList: undefined, courseProductRelation: undefined',
+      label: 'contractList: undefined, offer: undefined',
       contractList: undefined,
-      courseProductRelation: undefined,
+      offer: undefined,
     },
     {
-      label: 'contractList: 2 Contract, courseProductRelation: undefined',
+      label: 'contractList: 2 Contract, offer: undefined',
       contractList: ContractFactory().many(2),
-      courseProductRelation: undefined,
+      offer: undefined,
     },
     {
-      label: 'contractList: undefined, courseProductRelation: one CourseProductRelation',
+      label: 'contractList: undefined, offer: one Offer',
       contractList: undefined,
-      courseProductRelation: CourseProductRelationFactory().one(),
+      offer: OfferFactory().one(),
     },
   ])(
     'should implement AbstractContractFrame for organization and $label',
-    async ({ contractList, courseProductRelation }) => {
+    async ({ contractList, offer }) => {
       const organization = OrganizationFactory().one();
       const contracts = contractList || ContractFactory().many(2);
       const isOpen = faker.datatype.boolean();
 
-      const invitationLinkQueryString =
-        courseProductRelation && isCourseProductRelation(courseProductRelation)
-          ? `?course_product_relation_ids=${courseProductRelation.id}`
-          : '';
+      const invitationLinkQueryString = offer && isOffer(offer) ? `?offer_ids=${offer.id}` : '';
       const expectedUrls = {
         getInvitationLink: `https://joanie.endpoint/api/v1.0/organizations/${organization.id}/contracts-signature-link/${invitationLinkQueryString}`,
         checkSignature: `https://joanie.endpoint/api/v1.0/organizations/${organization.id}/contracts/?id=${contracts[0].id}&id=${contracts[1].id}`,
@@ -133,9 +126,7 @@ describe('OrganizationContractFrame', () => {
           <Wrapper client={client}>
             <OrganizationContractFrame
               organizationId={organization.id}
-              courseProductRelationIds={
-                courseProductRelation ? [courseProductRelation.id] : undefined
-              }
+              offerIds={offer ? [offer.id] : undefined}
               isOpen={isOpen}
               onDone={handleDone}
               onClose={handleClose}

package/js/components/ContractFrame/OrganizationContractFrame.tsx

@@ -4,17 +4,17 @@ import { useJoanieApi } from 'contexts/JoanieApiContext';
 import AbstractContractFrame, {
   AbstractProps,
 } from 'components/ContractFrame/AbstractContractFrame';
-import { Contract, CourseProductRelation } from 'types/Joanie';
+import { Contract, Offer } from 'types/Joanie';
 
 interface Props extends AbstractProps {
   contractIds?: Contract['id'][];
   organizationId: string;
-  courseProductRelationIds?: CourseProductRelation['id'][];
+  offerIds?: Offer['id'][];
 }
 
 const OrganizationContractFrame = ({
   organizationId,
-  courseProductRelationIds = [],
+  offerIds = [],
   contractIds,
   onDone,
   ...props
@@ -29,7 +29,7 @@ const OrganizationContractFrame = ({
      be signed. We need to keep track of these ids to check if all contracts have been signed.
      */
     const response = await api.organizations.contracts.getSignatureLinks({
-      course_product_relation_ids: courseProductRelationIds,
+      offer_ids: offerIds,
       organization_id: organizationId,
       contracts_ids: contractIds,
     });

package/js/components/CourseGlimpse/utils.ts

@@ -6,23 +6,18 @@ import {
   Course as RichieCourse,
   isRichieCourse,
 } from 'types/Course';
-import {
-  CourseListItem as JoanieCourse,
-  CourseProductRelationLight,
-  isCourseProductRelation,
-  ProductType,
-} from 'types/Joanie';
+import { CourseListItem as JoanieCourse, OfferLight, isOffer, ProductType } from 'types/Joanie';
 import { TeacherDashboardPaths } from 'widgets/Dashboard/utils/teacherDashboardPaths';
 import { CourseGlimpseCourse } from '.';
 
-const getCourseGlimpsePropsFromCourseProductRelation = (
-  courseProductRelation: CourseProductRelationLight,
+const getCourseGlimpsePropsFromOffer = (
+  offer: OfferLight,
   intl: IntlShape,
   organizationId?: string,
 ): CourseGlimpseCourse => {
   const courseRouteParams = {
-    courseId: courseProductRelation.course.id,
-    courseProductRelationId: courseProductRelation.id,
+    courseId: offer.course.id,
+    offerId: offer.id,
   };
   const courseRoute = organizationId
     ? generatePath(TeacherDashboardPaths.ORGANIZATION_PRODUCT, {
@@ -31,35 +26,27 @@ const getCourseGlimpsePropsFromCourseProductRelation = (
       })
     : generatePath(TeacherDashboardPaths.COURSE_PRODUCT, courseRouteParams);
   return {
-    id: courseProductRelation.id,
-    code: courseProductRelation.course.code,
-    title: courseProductRelation.product.title,
-    cover_image: courseProductRelation.course.cover
+    id: offer.id,
+    code: offer.course.code,
+    title: offer.product.title,
+    cover_image: offer.course.cover
       ? {
-          src: courseProductRelation.course.cover.src,
+          src: offer.course.cover.src,
         }
       : null,
     organization: {
-      title: courseProductRelation.organizations[0].title,
-      image: courseProductRelation.organizations[0].logo || null,
+      title: offer.organizations[0].title,
+      image: offer.organizations[0].logo || null,
     },
-    product_id: courseProductRelation.product.id,
+    product_id: offer.product.id,
     course_route: courseRoute,
-    state: courseProductRelation.product.state,
+    state: offer.product.state,
     certificate_offer:
-      courseProductRelation.product.type === ProductType.CERTIFICATE
-        ? CourseCertificateOffer.PAID
-        : null,
-    offer: courseProductRelation.product.type === ProductType.CREDENTIAL ? CourseOffer.PAID : null,
-    certificate_price:
-      courseProductRelation.product.type === ProductType.CERTIFICATE
-        ? courseProductRelation.product.price
-        : null,
-    price:
-      courseProductRelation.product.type === ProductType.CREDENTIAL
-        ? courseProductRelation.product.price
-        : null,
-    price_currency: courseProductRelation.product.price_currency,
+      offer.product.type === ProductType.CERTIFICATE ? CourseCertificateOffer.PAID : null,
+    offer: offer.product.type === ProductType.CREDENTIAL ? CourseOffer.PAID : null,
+    certificate_price: offer.product.type === ProductType.CERTIFICATE ? offer.product.price : null,
+    price: offer.product.type === ProductType.CREDENTIAL ? offer.product.price : null,
+    price_currency: offer.product.price_currency,
   };
 };
 
@@ -125,12 +112,12 @@ const getCourseGlimpsePropsFromJoanieCourse = (
 };
 
 export const getCourseGlimpseProps = (
-  course: RichieCourse | (JoanieCourse | CourseProductRelationLight),
+  course: RichieCourse | (JoanieCourse | OfferLight),
   intl?: IntlShape,
   organizationId?: string,
 ): CourseGlimpseCourse => {
-  if (isCourseProductRelation(course)) {
-    return getCourseGlimpsePropsFromCourseProductRelation(course, intl!, organizationId);
+  if (isOffer(course)) {
+    return getCourseGlimpsePropsFromOffer(course, intl!, organizationId);
   }
 
   if (isRichieCourse(course)) {

package/js/components/CourseGlimpseList/utils.ts

@@ -1,10 +1,10 @@
 import { IntlShape } from 'react-intl';
-import { CourseProductRelationLight, CourseListItem as JoanieCourse } from 'types/Joanie';
+import { OfferLight, CourseListItem as JoanieCourse } from 'types/Joanie';
 import { Course as RichieCourse } from 'types/Course';
 import { CourseGlimpseCourse, getCourseGlimpseProps } from 'components/CourseGlimpse';
 
 export const getCourseGlimpseListProps = (
-  courses: RichieCourse[] | (JoanieCourse | CourseProductRelationLight)[],
+  courses: RichieCourse[] | (JoanieCourse | OfferLight)[],
   intl?: IntlShape,
   organizationId?: string,
 ): CourseGlimpseCourse[] => {

package/js/components/PurchaseButton/index.tsx

@@ -42,7 +42,7 @@ const messages = defineMessages({
 
 interface PurchaseButtonPropsBase {
   product: Joanie.CredentialProduct | Joanie.CertificateProduct;
-  courseProductRelation?: Joanie.CourseProductRelation;
+  offer?: Joanie.Offer;
   isWithdrawable: boolean;
   disabled?: boolean;
   className?: string;
@@ -66,7 +66,7 @@ interface CertificatePurchaseButtonProps extends PurchaseButtonPropsBase {
 const PurchaseButton = ({
   product,
   course,
-  courseProductRelation,
+  offer,
   enrollment,
   isWithdrawable,
   organizations,
@@ -140,7 +140,7 @@ const PurchaseButton = ({
         {...saleTunnelModal}
         product={product}
         organizations={organizations}
-        courseProductRelation={courseProductRelation}
+        offer={offer}
         enrollment={enrollment}
         course={course}
         isWithdrawable={isWithdrawable}

package/js/components/SaleTunnel/GenericSaleTunnel.tsx

@@ -10,14 +10,7 @@ import {
 } from 'react';
 import { SaleTunnelSponsors } from 'components/SaleTunnel/Sponsors/SaleTunnelSponsors';
 import { SaleTunnelProps } from 'components/SaleTunnel/index';
-import {
-  Address,
-  CourseProductRelation,
-  CreditCard,
-  Order,
-  OrderState,
-  Product,
-} from 'types/Joanie';
+import { Address, Offer, CreditCard, Order, OrderState, Product } from 'types/Joanie';
 import useProductOrder from 'hooks/useProductOrder';
 import { SaleTunnelSuccess } from 'components/SaleTunnel/SaleTunnelSuccess';
 import WebAnalyticsAPIHandler from 'api/web-analytics';
@@ -33,7 +26,7 @@ export interface SaleTunnelContextType {
   order?: Order;
   product: Product;
   webAnalyticsEventKey: string;
-  relation?: CourseProductRelation;
+  offer?: Offer;
 
   // internal
   step: SaleTunnelStep;
@@ -121,7 +114,7 @@ export const GenericSaleTunnel = (props: GenericSaleTunnelProps) => {
       webAnalyticsEventKey: props.eventKey,
       order,
       product: props.product,
-      relation: props.courseProductRelation,
+      offer: props.offer,
       props,
       billingAddress,
       setBillingAddress,

package/js/components/SaleTunnel/SaleTunnelInformation/index.tsx

@@ -8,6 +8,7 @@ import useOpenEdxProfile from 'hooks/useOpenEdxProfile';
 import { usePaymentSchedule } from 'hooks/usePaymentSchedule';
 import { Spinner } from 'components/Spinner';
 import WithdrawRightCheckbox from 'components/SaleTunnel/WithdrawRightCheckbox';
+import { ProductType } from 'types/Joanie';
 
 const messages = defineMessages({
   title: {
@@ -54,6 +55,7 @@ const messages = defineMessages({
 });
 
 export const SaleTunnelInformation = () => {
+  const { product } = useSaleTunnelContext();
   return (
     <div className="sale-tunnel__main__column sale-tunnel__information">
       <div>
@@ -70,7 +72,7 @@ export const SaleTunnelInformation = () => {
         </div>
       </div>
       <div>
-        <PaymentScheduleBlock />
+        {product.type === ProductType.CREDENTIAL && <PaymentScheduleBlock />}
         <Total />
         <WithdrawRightCheckbox />
       </div>
@@ -99,7 +101,7 @@ const Email = () => {
 };
 
 const Total = () => {
-  const { product, relation } = useSaleTunnelContext();
+  const { product, offer } = useSaleTunnelContext();
   return (
     <div className="sale-tunnel__total">
       <div className="sale-tunnel__total__amount mt-t" data-testid="sale-tunnel__total__amount">
@@ -108,7 +110,7 @@ const Total = () => {
         </div>
         <div className="block-title">
           <FormattedNumber
-            value={relation?.discounted_price || product.price}
+            value={offer?.rules.discounted_price || product.price}
             style="currency"
             currency={product.price_currency}
           />

package/js/components/SaleTunnel/index.full-process.spec.tsx

@@ -15,7 +15,7 @@ import CourseProductItem from 'widgets/SyllabusCourseRunsList/components/CourseP
 import {
   AddressFactory,
   ContractFactory,
-  CourseProductRelationFactory,
+  OfferFactory,
   CredentialOrderFactory,
   CreditCardFactory,
   PaymentFactory,
@@ -99,7 +99,7 @@ describe('SaleTunnel', () => {
      */
     const course = PacedCourseFactory().one();
     const product = ProductFactory().one();
-    const relation = CourseProductRelationFactory({
+    const offer = OfferFactory({
       course,
       product,
       is_withdrawable: false,
@@ -108,7 +108,7 @@ describe('SaleTunnel', () => {
 
     fetchMock.get(
       `https://joanie.endpoint/api/v1.0/courses/${course.code}/products/${product.id}/`,
-      relation,
+      offer,
     );
     fetchMock.get(
       `https://joanie.endpoint/api/v1.0/courses/${course.code}/products/${product.id}/payment-schedule/`,

package/js/components/SaleTunnel/index.spec.tsx

@@ -15,7 +15,7 @@ import {
   AddressFactory,
   CertificateOrderFactory,
   CertificateProductFactory,
-  CourseProductRelationFactory,
+  OfferFactory,
   CredentialOrderFactory,
   CredentialProductFactory,
   CreditCardFactory,
@@ -182,7 +182,9 @@ describe.each([
     nbApiCalls += 1; // useProductOrder call.
     nbApiCalls += 1; // get user account call.
     nbApiCalls += 1; // get user preferences call.
-    nbApiCalls += 1; // product payment-schedule call
+    if (product.type === ProductType.CREDENTIAL) {
+      nbApiCalls += 1; // product payment-schedule call
+    }
     await waitFor(() => expect(fetchMock.calls()).toHaveLength(nbApiCalls));
 
     const user = userEvent.setup({ delay: null });
@@ -269,7 +271,9 @@ describe.each([
     nbApiCalls += 1; // useProductOrder get order with filters
     nbApiCalls += 1; // get user account call.
     nbApiCalls += 1; // get user preferences call.
-    nbApiCalls += 1; // get product payment schedule.
+    if (product.type === ProductType.CREDENTIAL) {
+      nbApiCalls += 1; // get product payment schedule.
+    }
     await waitFor(() => expect(fetchMock.calls()).toHaveLength(nbApiCalls));
 
     const user = userEvent.setup({ delay: null });
@@ -403,36 +407,41 @@ describe.each([
       queryOptions: { client: createTestQueryClient({ user: richieUser }) },
     });
 
-    await screen.findByRole('heading', {
-      level: 4,
-      name: 'Payment schedule',
-    });
+    if (product.type === ProductType.CREDENTIAL) {
+      await screen.findByRole('heading', {
+        level: 4,
+        name: 'Payment schedule',
+      });
 
-    const scheduleTable = screen.getByRole('table');
-    const scheduleTableRows = within(scheduleTable).getAllByRole('row');
-    expect(scheduleTableRows).toHaveLength(schedule.length);
+      const scheduleTable = screen.getByRole('table');
+      const scheduleTableRows = within(scheduleTable).getAllByRole('row');
+      expect(scheduleTableRows).toHaveLength(schedule.length);
 
-    scheduleTableRows.forEach((row, index) => {
-      const installment = schedule[index];
-      // A first column should show the installment index
-      within(row).getByRole('cell', {
-        name: (index + 1).toString(),
-      });
-      // A 2nd column should show the installment amount
-      within(row).getByRole('cell', {
-        name: formatPrice(installment.amount, installment.currency),
-      });
-      // A 3rd column should show the installment withdraw date
-      within(row).getByRole('cell', {
-        name: `Withdrawn on ${intl.formatDate(installment.due_date, {
-          ...DEFAULT_DATE_FORMAT,
-        })}`,
-      });
-      // A 4th column should show the installment state
-      within(row).getByRole('cell', {
-        name: StringHelper.capitalizeFirst(installment.state.replace('_', ' '))!,
+      scheduleTableRows.forEach((row, index) => {
+        const installment = schedule[index];
+        // A first column should show the installment index
+        within(row).getByRole('cell', {
+          name: (index + 1).toString(),
+        });
+        // A 2nd column should show the installment amount
+        within(row).getByRole('cell', {
+          name: formatPrice(installment.amount, installment.currency),
+        });
+        // A 3rd column should show the installment withdraw date
+        within(row).getByRole('cell', {
+          name: `Withdrawn on ${intl.formatDate(installment.due_date, {
+            ...DEFAULT_DATE_FORMAT,
+          })}`,
+        });
+        // A 4th column should show the installment state
+        within(row).getByRole('cell', {
+          name: StringHelper.capitalizeFirst(installment.state.replace('_', ' '))!,
+        });
       });
-    });
+    } else {
+      expect(screen.queryByRole('heading', { level: 4, name: 'Payment schedule' })).toBeNull();
+      expect(screen.queryByRole('table')).toBeNull();
+    }
 
     const $totalAmount = screen.getByTestId('sale-tunnel__total__amount');
     expect($totalAmount).toHaveTextContent(
@@ -444,15 +453,17 @@ describe.each([
     const intl = createIntl({ locale: 'en' });
     const schedule = PaymentInstallmentFactory().many(2);
 
-    const relation = CourseProductRelationFactory({
+    const offer = OfferFactory({
       product: ProductFactory({
         price: 840,
         price_currency: 'EUR',
       }).one(),
-      discounted_price: 800,
-      discount_rate: 0.3,
+      rules: {
+        discounted_price: 800,
+        discount_rate: 0.3,
+      },
     }).one();
-    const { product } = relation;
+    const { product } = offer;
 
     fetchMock
       .get(
@@ -464,45 +475,47 @@ describe.each([
         schedule,
       );
 
-    render(<Wrapper product={product} courseProductRelation={relation} isWithdrawable={true} />, {
+    render(<Wrapper product={product} offer={offer} isWithdrawable={true} />, {
       queryOptions: { client: createTestQueryClient({ user: richieUser }) },
     });
 
-    await screen.findByRole('heading', {
-      level: 4,
-      name: 'Payment schedule',
-    });
+    if (product.type === ProductType.CREDENTIAL) {
+      await screen.findByRole('heading', { level: 4, name: 'Payment schedule' });
 
-    const scheduleTable = screen.getByRole('table');
-    const scheduleTableRows = within(scheduleTable).getAllByRole('row');
-    expect(scheduleTableRows).toHaveLength(schedule.length);
+      const scheduleTable = screen.getByRole('table');
+      const scheduleTableRows = within(scheduleTable).getAllByRole('row');
+      expect(scheduleTableRows).toHaveLength(schedule.length);
 
-    scheduleTableRows.forEach((row, index) => {
-      const installment = schedule[index];
-      // A first column should show the installment index
-      within(row).getByRole('cell', {
-        name: (index + 1).toString(),
-      });
-      // A 2nd column should show the installment amount
-      within(row).getByRole('cell', {
-        name: formatPrice(installment.amount, installment.currency),
-      });
-      // A 3rd column should show the installment withdraw date
-      within(row).getByRole('cell', {
-        name: `Withdrawn on ${intl.formatDate(installment.due_date, {
-          ...DEFAULT_DATE_FORMAT,
-        })}`,
-      });
-      // A 4th column should show the installment state
-      within(row).getByRole('cell', {
-        name: StringHelper.capitalizeFirst(installment.state.replace('_', ' '))!,
+      scheduleTableRows.forEach((row, index) => {
+        const installment = schedule[index];
+        // A first column should show the installment index
+        within(row).getByRole('cell', {
+          name: (index + 1).toString(),
+        });
+        // A 2nd column should show the installment amount
+        within(row).getByRole('cell', {
+          name: formatPrice(installment.amount, installment.currency),
+        });
+        // A 3rd column should show the installment withdraw date
+        within(row).getByRole('cell', {
+          name: `Withdrawn on ${intl.formatDate(installment.due_date, {
+            ...DEFAULT_DATE_FORMAT,
+          })}`,
+        });
+        // A 4th column should show the installment state
+        within(row).getByRole('cell', {
+          name: StringHelper.capitalizeFirst(installment.state.replace('_', ' '))!,
+        });
       });
-    });
+    } else {
+      expect(screen.queryByRole('heading', { level: 4, name: 'Payment schedule' })).toBeNull();
+      expect(screen.queryByRole('table')).toBeNull();
+    }
 
     const $totalAmount = screen.getByTestId('sale-tunnel__total__amount');
     expect($totalAmount).toHaveTextContent(
       'Total' +
-        formatPrice(relation!.discounted_price!, product.price_currency).replace(
+        formatPrice(offer!.rules.discounted_price!, product.price_currency).replace(
           /(\u202F|\u00a0)/g,
           ' ',
         ),

package/js/components/SaleTunnel/index.tsx

@@ -2,7 +2,7 @@ import { ModalProps } from '@openfun/cunningham-react';
 import {
   CertificateProduct,
   CourseLight,
-  CourseProductRelation,
+  Offer,
   CredentialProduct,
   Enrollment,
   Order,
@@ -16,7 +16,7 @@ import { PacedCourse } from 'types';
 
 export interface SaleTunnelProps extends Pick<ModalProps, 'isOpen' | 'onClose'> {
   product: Product;
-  courseProductRelation?: CourseProductRelation;
+  offer?: Offer;
   organizations?: Organization[];
   isWithdrawable: boolean;
   course?: PacedCourse | CourseLight;

package/js/components/TeacherDashboardCourseList/index.spec.tsx

@@ -1,7 +1,7 @@
 import { screen } from '@testing-library/react';
 import fetchMock from 'fetch-mock';
 import { RichieContextFactory as mockRichieContextFactory } from 'utils/test/factories/richie';
-import { CourseListItemFactory, CourseProductRelationFactory } from 'utils/test/factories/joanie';
+import { CourseListItemFactory, OfferFactory } from 'utils/test/factories/joanie';
 import { render } from 'utils/test/render';
 import { setupJoanieSession } from 'utils/test/wrappers/JoanieAppWrapper';
 import { expectNoSpinner, expectSpinner } from 'utils/test/expectSpinner';
@@ -35,7 +35,7 @@ describe('components/TeacherDashboardCourseList', () => {
   });
 
   it('should render loading more state', async () => {
-    const trainings = CourseProductRelationFactory().many(2);
+    const trainings = OfferFactory().many(2);
     const courses = CourseListItemFactory().many(2);
     const courseAndProductList = [...courses, ...trainings];
 
@@ -60,7 +60,7 @@ describe('components/TeacherDashboardCourseList', () => {
   });
 
   it('should render courses and products list', async () => {
-    const trainings = CourseProductRelationFactory().many(2);
+    const trainings = OfferFactory().many(2);
     const courses = CourseListItemFactory().many(2);
     const courseAndProductList = [...courses, ...trainings];
 

package/js/components/TeacherDashboardCourseList/index.tsx

@@ -6,7 +6,7 @@ import { CourseGlimpseList, getCourseGlimpseListProps } from 'components/CourseG
 import { Spinner } from 'components/Spinner';
 import context from 'utils/context';
 import { useIntersectionObserver } from 'hooks/useIntersectionObserver';
-import { CourseListItem, CourseProductRelationLight } from 'types/Joanie';
+import { CourseListItem, OfferLight } from 'types/Joanie';
 import Banner from 'components/Banner';
 
 const messages = defineMessages({
@@ -31,7 +31,7 @@ interface TeacherDashboardCourseListProps {
   titleTranslated?: string;
   organizationId?: string;
   loadMore: () => void;
-  courseAndProductList?: (CourseListItem | CourseProductRelationLight)[];
+  courseAndProductList?: (CourseListItem | OfferLight)[];
   isLoadingMore?: boolean;
   hasMore?: boolean;
   isNewSearchLoading?: boolean;