richie 2.25.0b2.dev44__py2.py3-none-any.whl → 2.25.0b2.dev49__py2.py3-none-any.whl

frontend/js/components/TeacherDashboardCourseList/index.tsx

@@ -53,6 +53,16 @@ const TeacherDashboardCourseList = ({
       {titleTranslated && (
         <h2 className="dashboard-course-list__title dashboard__page_title">{titleTranslated}</h2>
       )}
+      {courseAndProductList.length > 0 ? (
+        <CourseGlimpseList
+          courses={getCourseGlimpseListProps(courseAndProductList, intl, organizationId)}
+          context={context}
+          className="dashboard__course-glimpse-list"
+        />
+      ) : (
+        <FormattedMessage {...messages.emptyList} />
+      )}
+
       {isLoading && (
         <Spinner aria-labelledby="loading-courses-data">
           <span id="loading-courses-data">
@@ -60,28 +70,17 @@ const TeacherDashboardCourseList = ({
           </span>
         </Spinner>
       )}
-      {!isLoading &&
-        (courseAndProductList.length > 0 ? (
-          <>
-            <CourseGlimpseList
-              courses={getCourseGlimpseListProps(courseAndProductList, intl, organizationId)}
-              context={context}
-              className="dashboard__course-glimpse-list"
-            />
-            {hasMore && (
-              <Button
-                onClick={() => next()}
-                disabled={isLoading}
-                ref={loadMoreButtonRef}
-                color="tertiary"
-              >
-                <FormattedMessage {...messages.loadMore} />
-              </Button>
-            )}
-          </>
-        ) : (
-          <FormattedMessage {...messages.emptyList} />
-        ))}
+
+      {hasMore && (
+        <Button
+          onClick={() => next()}
+          disabled={isLoading}
+          ref={loadMoreButtonRef}
+          color="tertiary"
+        >
+          <FormattedMessage {...messages.loadMore} />
+        </Button>
+      )}
     </div>
   );
 };

frontend/js/types/Joanie.ts

@@ -1,4 +1,4 @@
-import type { CourseState } from 'types';
+import type { CourseState, OpenEdXEnrollment } from 'types';
 import type { Maybe, Nullable } from 'types/utils';
 import { Resource, ResourcesQuery } from 'hooks/useResources';
 import { OrderResourcesQuery } from 'hooks/useOrders';
@@ -190,8 +190,23 @@ export interface Enrollment {
   created_on: string;
   orders: OrderEnrollment[];
   product_relations: CourseProductRelation[];
-  certificate_id?: string;
-}
+  certificate_id: Nullable<string>;
+}
+export const isEnrollment = (obj: unknown | Enrollment | OpenEdXEnrollment): obj is Enrollment => {
+  if (!obj || typeof obj !== 'object') {
+    return false;
+  }
+  return (
+    'is_active' in obj &&
+    'state' in obj &&
+    'course_run' in obj &&
+    'was_created_by_order' in obj &&
+    'created_on' in obj &&
+    'orders' in obj &&
+    'product_relations' in obj &&
+    'certificate_id' in obj
+  );
+};
 
 export interface EnrollmentLight {
   id: string;

frontend/js/types/index.ts

@@ -69,4 +69,5 @@ export interface OpenEdXEnrollment {
  * Use an unknown type to make sure we do not depend on any LMS-specific fields
  * on enrollment objects, just use HTTP response codes.
  */
+// TODO(rlecellier): rename into UnknownEnrollment
 export type Enrollment = unknown;

frontend/js/utils/test/factories/joanie.ts

@@ -81,6 +81,7 @@ export const EnrollmentFactory = factory((): Enrollment => {
     was_created_by_order: false,
     created_on: faker.date.past({ years: 1 }).toISOString(),
     orders: [],
+    certificate_id: null,
   };
 });
 

frontend/js/widgets/Dashboard/components/DashboardItem/stories.mock.ts

@@ -26,4 +26,5 @@ export const enrollment: Enrollment = {
     languages: ['en'],
   },
   product_relations: [],
+  certificate_id: null,
 };

frontend/js/widgets/SyllabusCourseRunsList/components/CourseRunEnrollment/index.joanie.spec.tsx

@@ -3,7 +3,9 @@ import fetchMock from 'fetch-mock';
 import { QueryClientProvider } from '@tanstack/react-query';
 import { IntlProvider } from 'react-intl';
 
+import { faker } from '@faker-js/faker';
 import { Deferred } from 'utils/test/deferred';
+import { EnrollmentFactory as JoanieEnrollment } from 'utils/test/factories/joanie';
 import {
   CourseRunFactory,
   RichieContextFactory as mockRichieContextFactory,
@@ -12,6 +14,7 @@ import {
 import { SessionProvider } from 'contexts/SessionContext';
 import { createTestQueryClient } from 'utils/test/createTestQueryClient';
 import { HttpStatusCode } from 'utils/errors/HttpError';
+import { Priority } from 'types';
 import CourseRunEnrollment from './index';
 
 jest.mock('utils/errors/handle');
@@ -96,10 +99,11 @@ describe('<CourseRunEnrollment /> with joanie backend ', () => {
   it('shows an error message when enrollment get request failed', async () => {
     const user = UserFactory().one();
     const courseRun = CourseRunFactory().one();
-    courseRun.resource_link = `https://joanie.endpoint/api/v1.0/course-runs/${courseRun.id}`;
+    const joanieEnrollmentId = faker.string.uuid();
+    courseRun.resource_link = `https://joanie.endpoint/api/v1.0/course-runs/${joanieEnrollmentId}`;
 
     fetchMock.get(
-      `${endpoint}/api/v1.0/enrollments/?course_run=${courseRun.id}`,
+      `${endpoint}/api/v1.0/enrollments/?course_run_id=${joanieEnrollmentId}`,
       HttpStatusCode.INTERNAL_SERVER_ERROR,
     );
 
@@ -118,6 +122,45 @@ describe('<CourseRunEnrollment /> with joanie backend ', () => {
     await screen.findByText('Enrollment fetching failed');
   });
 
+  it('shows a link to the course if the user is already enrolled', async () => {
+    // Joanie session requests
+    let nbApiCalls = 3;
+    const user = UserFactory().one();
+    const joanieEnrollment = JoanieEnrollment({ is_active: true }).one();
+    const courseRun = CourseRunFactory().one();
+    courseRun.resource_link = `https://joanie.endpoint/api/v1.0/course-runs/${joanieEnrollment.course_run.id}/`;
+    courseRun.state.priority = Priority.ONGOING_OPEN;
+
+    fetchMock.get(
+      `${endpoint}/api/v1.0/enrollments/?course_run_id=${joanieEnrollment.course_run.id}`,
+      {
+        count: 1,
+        next: null,
+        previous: null,
+        results: [joanieEnrollment],
+      },
+    );
+    nbApiCalls += 1;
+
+    await act(async () => {
+      render(
+        <QueryClientProvider client={createTestQueryClient({ user })}>
+          <IntlProvider locale="en">
+            <SessionProvider>
+              <CourseRunEnrollment courseRun={courseRun} />
+            </SessionProvider>
+          </IntlProvider>
+        </QueryClientProvider>,
+      );
+    });
+
+    const $goToCourseButton = await screen.findByRole('link', { name: 'Go to course' });
+    expect($goToCourseButton).toBeInTheDocument();
+    expect($goToCourseButton).toHaveAttribute('href', joanieEnrollment.course_run.resource_link);
+    expect(screen.getByText('You are enrolled in this course run')).toBeInTheDocument();
+    expect(fetchMock.calls()).toHaveLength(nbApiCalls);
+  });
+
   it('shows an "Unenroll" text and allows the user to unenroll', async () => {
     const user = UserFactory().one();
     const courseRun = CourseRunFactory().one();

frontend/js/widgets/SyllabusCourseRunsList/components/CourseRunEnrollment/index.tsx

@@ -1,4 +1,4 @@
-import React, { useCallback, useEffect, useReducer } from 'react';
+import React, { useCallback, useEffect, useMemo, useReducer } from 'react';
 import { defineMessages, FormattedMessage } from 'react-intl';
 import c from 'classnames';
 import { Button } from '@openfun/cunningham-react';
@@ -12,6 +12,7 @@ import { HttpError } from 'utils/errors/HttpError';
 import useCourseEnrollment from 'widgets/SyllabusCourseRunsList/hooks/useCourseEnrollment';
 import { CourseRunUnenrollButton } from 'widgets/SyllabusCourseRunsList/components/CourseRunEnrollment/CourseRunUnenrollmentButton';
 import useDateRelative from 'hooks/useDateRelative';
+import { isEnrollment as isJoanieEnrollment } from 'types/Joanie';
 
 const messages = defineMessages({
   enroll: {
@@ -181,9 +182,8 @@ const reducer = ({ step, context }: ReducerState, action: ReducerAction): Reduce
 
 const CourseRunEnrollment: React.FC<CourseRunEnrollmentProps> = (props) => {
   const { user, login } = useSession();
-  const { enrollmentIsActive, setEnrollment, canUnenroll, states } = useCourseEnrollment(
-    props.courseRun.resource_link,
-  );
+  const { enrollment, enrollmentIsActive, setEnrollment, canUnenroll, states } =
+    useCourseEnrollment(props.courseRun.resource_link);
   const startDate = new Date(props.courseRun.start);
   const isStarted = new Date() > startDate;
   const relativeStartDate = useDateRelative(startDate);
@@ -226,6 +226,16 @@ const CourseRunEnrollment: React.FC<CourseRunEnrollmentProps> = (props) => {
     [courseRun, currentUser, dispatch, enrollmentIsActive],
   );
 
+  const LmsCourseLink = useMemo(() => {
+    if (states.isLoading) {
+      return null;
+    }
+    if (isJoanieEnrollment(enrollment)) {
+      return enrollment.course_run.resource_link;
+    }
+    return courseRun.resource_link;
+  }, [courseRun, enrollment]);
+
   useEffect(() => {
     dispatch({
       payload: { currentUser: user, isEnrolled: enrollmentIsActive },
@@ -303,7 +313,8 @@ const CourseRunEnrollment: React.FC<CourseRunEnrollmentProps> = (props) => {
       return isStarted ? (
         <div>
           <Button
-            href={courseRun.resource_link}
+            href={LmsCourseLink === null ? '#' : LmsCourseLink}
+            disabled={LmsCourseLink === null}
             className="course-run-enrollment__cta"
             fullWidth={true}
           >

frontend/js/widgets/SyllabusCourseRunsList/hooks/useCourseEnrollment/index.ts

@@ -21,22 +21,18 @@ const useCourseEnrollment = (resourceLink: string) => {
   const queryClient = useQueryClient();
   const EnrollmentAPI = EnrollmentApiInterface(resourceLink);
 
-  const [{ data: enrollment, isError, isLoading }, queryKey] = useSessionQuery(
+  const [{ data: enrollment, isError, isLoading: isEnrollmentLoading }, queryKey] = useSessionQuery(
     ['enrollment', resourceLink],
     async () => {
       return EnrollmentAPI.get(resourceLink, user!);
     },
   );
 
-  const [{ data: isActive, refetch: refetchIsActive }] = useSessionQuery(
-    [...queryKey, 'is_active'],
-    async () => EnrollmentAPI.isEnrolled(enrollment),
-    {
+  const [{ data: isActive, refetch: refetchIsActive, isLoading: isActiveLoading }] =
+    useSessionQuery([...queryKey, 'is_active'], async () => EnrollmentAPI.isEnrolled(enrollment), {
       // Enrollment is null if it has been fetched
-      enabled: !!user && enrollment !== undefined && !isLoading,
-    },
-  );
-
+      enabled: !!user && enrollment !== undefined && !isEnrollmentLoading,
+    });
   const { mutateAsync } = useSessionMutation({
     mutationFn: (activeEnrollment: boolean = true) =>
       EnrollmentAPI.set(resourceLink, user!, enrollment, activeEnrollment),
@@ -61,6 +57,7 @@ const useCourseEnrollment = (resourceLink: string) => {
       errors: {
         get: isError,
       },
+      isLoading: isEnrollmentLoading || isActiveLoading,
     },
   };
 };

richie/__init__.py

@@ -1,6 +1,7 @@
 """
 Richie: a FUN portal for Open edX
 """
+
 from importlib_metadata import version
 
 

richie/apps/core/admin.py

@@ -1,4 +1,5 @@
 """Utility functions for admin classes."""
+
 from django.urls import reverse
 from django.utils.html import format_html
 

richie/apps/core/context_processors.py

@@ -1,6 +1,7 @@
 """
 Template context processors
 """
+
 import json
 from collections import OrderedDict
 from urllib.parse import urlparse
@@ -126,9 +127,9 @@ def site_metas(request: HttpRequest):
     context["FRONTEND_CONTEXT"] = json.dumps(context["FRONTEND_CONTEXT"])
 
     if getattr(settings, "RICHIE_MINIMUM_COURSE_RUNS_ENROLLMENT_COUNT", None):
-        context[
-            "RICHIE_MINIMUM_COURSE_RUNS_ENROLLMENT_COUNT"
-        ] = settings.RICHIE_MINIMUM_COURSE_RUNS_ENROLLMENT_COUNT
+        context["RICHIE_MINIMUM_COURSE_RUNS_ENROLLMENT_COUNT"] = (
+            settings.RICHIE_MINIMUM_COURSE_RUNS_ENROLLMENT_COUNT
+        )
 
     return context
 

richie/apps/core/defaults.py

@@ -1,4 +1,5 @@
 """Default settings for the core app of Richie."""
+
 from django.conf import global_settings, settings
 from django.core.exceptions import ImproperlyConfigured
 from django.utils.functional import lazy

richie/apps/core/factories.py

@@ -1,6 +1,7 @@
 """
 Core factories
 """
+
 import io
 import os
 import random

richie/apps/core/fields/duration.py

@@ -2,6 +2,7 @@
 A custom field that defines a duration as a number of pre-defined time units
 (e.g 1 hour, 5 hours or 3 days)
 """
+
 from django import forms
 from django.core import checks, exceptions
 from django.db import models

richie/apps/core/fields/effort.py

@@ -2,6 +2,7 @@
 A custom field that defines efforts as a number of pre-defined time units per reference time unit
 (e.g 2 hours/day, 5 hours/week or 3 days/months).
 """
+
 from django import forms
 from django.core import checks, exceptions
 from django.db import models

richie/apps/core/fields/multiselect.py

@@ -1,4 +1,5 @@
 """A multi select field where the array of values is stored as a comma separated string."""
+
 from django.core import checks, exceptions, validators
 from django.db import models
 from django.forms import MultipleChoiceField, widgets

richie/apps/core/helpers.py

@@ -1,6 +1,7 @@
 """
 Helpers that can be useful throughout the whole project
 """
+
 from functools import reduce
 from operator import or_
 

richie/apps/core/models.py

@@ -1,4 +1,5 @@
 """Define a custom manager for page extensions"""
+
 from django.apps import apps
 from django.db import models
 from django.utils import translation

richie/apps/core/templatetags/feature_flags.py

@@ -1,4 +1,5 @@
 """Custom template tags related to FEATURES settings."""
+
 from django import template
 from django.conf import settings
 

richie/apps/core/templatetags/joanie.py

@@ -1,4 +1,5 @@
 """Custom template tags related to Joanie."""
+
 from django import template
 
 from richie.apps.courses.lms import LMSHandler

richie/apps/core/templatetags/language_name.py

@@ -1,4 +1,5 @@
 """Simple tag related with language and i18n."""
+
 from django import template
 from django.conf import settings
 from django.utils import translation

richie/apps/core/templatetags/rfc_5646_locale.py

@@ -1,4 +1,5 @@
 """Custom template tags for the search application of Richie."""
+
 from django import template
 from django.core.exceptions import ImproperlyConfigured
 

richie/apps/core/tests/utils.py

@@ -1,6 +1,7 @@
 """
 Useful stuff for testing
 """
+
 from cms.test_utils.testcases import CMSTestCase
 
 

richie/apps/core/views/error.py

@@ -1,4 +1,5 @@
 """Error views"""
+
 from django.shortcuts import render
 from django.utils.translation import gettext_lazy as _
 

richie/apps/courses/admin.py

@@ -1,6 +1,7 @@
 """
 Courses application admin
 """
+
 from itertools import chain
 from operator import itemgetter
 

richie/apps/courses/api.py

@@ -1,6 +1,7 @@
 """
 API endpoints for the courses app.
 """
+
 from django.conf import settings
 from django.db.models import Q
 

richie/apps/courses/cms_toolbars.py

@@ -1,6 +1,7 @@
 """
 Toolbar extension for the courses application
 """
+
 from django.utils.text import capfirst
 from django.utils.translation import gettext_lazy as _
 

richie/apps/courses/cms_wizards.py

@@ -1,6 +1,7 @@
 """
 CMS Wizard to add a course page
 """
+
 from django import forms
 from django.conf import settings
 from django.core import validators

richie/apps/courses/defaults.py

@@ -1,6 +1,7 @@
 """
 Settings specific to the courses application.
 """
+
 from django.conf import settings
 from django.utils.translation import gettext_lazy as _
 

richie/apps/courses/factories.py

@@ -1,6 +1,7 @@
 """
 Courses factories
 """
+
 import random
 from collections import namedtuple
 from datetime import datetime, timedelta

richie/apps/courses/fields.py

@@ -1,6 +1,7 @@
 """
 Custom fields for the course application
 """
+
 from datetime import datetime, time
 
 from django.forms import SplitDateTimeField

richie/apps/courses/helpers.py

@@ -1,6 +1,7 @@
 """
 Helpers that can be useful throughout Richie's courses app
 """
+
 from django.core.exceptions import PermissionDenied
 from django.utils import timezone
 from django.utils.text import slugify

richie/apps/courses/lms/__init__.py

@@ -1,4 +1,5 @@
 """LMS handler to select and return the right LMS backend for each url."""
+
 import re
 
 from django.conf import settings

richie/apps/courses/lms/base.py

@@ -1,6 +1,7 @@
 """
 Base backend to connect richie with an LMS
 """
+
 from django.conf import settings
 
 from ..models.course import CourseRunSyncMode

richie/apps/courses/lms/edx.py

@@ -1,6 +1,7 @@
 """
 Backend to connect Open edX richie with an LMS
 """
+
 import logging
 import re
 

richie/apps/courses/lms/joanie.py

@@ -1,6 +1,7 @@
 """
 Backend to connect Richie with Joanie
 """
+
 import re
 
 from .base import BaseLMSBackend

richie/apps/courses/management/commands/richie_init.py

@@ -1,4 +1,5 @@
 """Create_demo_site management command."""
+
 import logging
 
 from django.contrib.sites.models import Site

richie/apps/courses/migrations/0031_auto_20210811_1234.py

@@ -12,9 +12,11 @@ def normalize_course_code(apps, schema_editor):
 
     for course in Course.objects.only("id", "code").iterator():
         Course.objects.filter(id=course.id).update(
-            code=slugify(course.code, allow_unicode=True).upper()
-            if course.code
-            else None
+            code=(
+                slugify(course.code, allow_unicode=True).upper()
+                if course.code
+                else None
+            )
         )
 
 

richie/apps/courses/models/__init__.py

@@ -1,4 +1,5 @@
 """Make models.py a module containing one file per model as it was getting too long."""
+
 # flake8: noqa
 # pylint: disable=wildcard-import
 

richie/apps/courses/models/blog.py

@@ -1,6 +1,7 @@
 """
 Declare and configure the models for the blog part
 """
+
 from django.db import models
 from django.utils import translation
 from django.utils.translation import gettext_lazy as _

richie/apps/courses/models/category.py

@@ -1,6 +1,7 @@
 """
 Declare and configure the models for the courses application
 """
+
 from django.conf import settings
 from django.db import models
 from django.utils.translation import gettext_lazy as _

richie/apps/courses/models/course.py

@@ -1,6 +1,7 @@
 """
 Declare and configure the models for the courses application
 """
+
 # pylint: disable=too-many-lines
 from collections.abc import Mapping
 from datetime import MAXYEAR, datetime

richie/apps/courses/models/organization.py

@@ -1,6 +1,7 @@
 """
 Declare and configure the models for the courses application
 """
+
 from django.conf import settings
 from django.core.exceptions import ValidationError
 from django.db import models

richie/apps/courses/models/person.py

@@ -1,6 +1,7 @@
 """
 Declare and configure the model for the person application
 """
+
 from django.db import models
 from django.utils.translation import gettext_lazy as _
 

richie/apps/courses/models/program.py

@@ -1,6 +1,7 @@
 """
 Declare and configure the models for the program part
 """
+
 from django.db import models
 from django.utils.translation import gettext_lazy as _
 

richie/apps/courses/models/role.py

@@ -1,6 +1,7 @@
 """
 Declare and configure the models for the courses application
 """
+
 from django.contrib.auth.models import Group
 from django.db import models
 from django.utils.translation import gettext_lazy as _

richie/apps/courses/serializers.py

@@ -1,4 +1,5 @@
 """Serializers for Richie's courses app."""
+
 from django.utils.functional import lazy
 
 from rest_framework import serializers

richie/apps/courses/settings/__init__.py

@@ -16,6 +16,7 @@ Otherwise, you can just use the usual Django pattern in your settings.py file:
     from richie.apps.courses.settings import *
     ```
 """
+
 from django.utils.translation import gettext_lazy as _
 
 

richie/apps/courses/settings/mixins.py

@@ -1,4 +1,5 @@
 """Django Configuration mixins for the Richie courses app."""
+
 from configurations.utils import uppercase_attributes
 
 from .. import settings

richie/apps/courses/templatetags/category_tags.py

@@ -1,4 +1,5 @@
 """Richie template tags to manipulate categories."""
+
 from django import template
 from django.utils import translation