Prefix course titles in dashboard's course section

hypothesis · Jul 25, 2024 · 529d96c · 529d96c
1 parent bf6d736
commit 529d96c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/lms/static/scripts/frontend_apps/components/dashboard/CourseActivity.tsx b/lms/static/scripts/frontend_apps/components/dashboard/CourseActivity.tsx
@@ -37,6 +37,7 @@ export default function CourseActivity() {
     }),
   );
 
+  const title = `Course: ${course.data?.title}`;
   const rows: AssignmentsTableRow[] = useMemo(
     () =>
       (assignments.data?.assignments ?? []).map(
@@ -60,12 +61,12 @@ export default function CourseActivity() {
         <h2 className="text-lg text-brand font-semibold" data-testid="title">
           {course.isLoading && 'Loading...'}
           {course.error && 'Could not load course title'}
-          {course.data && course.data.title}
+          {course.data && title}
         </h2>
       </div>
       <OrderableActivityTable
         loading={assignments.isLoading}
-        title={course.data?.title ?? 'Loading...'}
+        title={course.isLoading ? 'Loading...' : title}
         emptyMessage={
           assignments.error
             ? 'Could not load assignments'

diff --git a/lms/static/scripts/frontend_apps/components/dashboard/test/CourseActivity-test.js b/lms/static/scripts/frontend_apps/components/dashboard/test/CourseActivity-test.js
@@ -105,7 +105,7 @@ describe('CourseActivity', () => {
     const wrapper = createComponent();
     const titleElement = wrapper.find('[data-testid="title"]');
     const tableElement = wrapper.find('OrderableActivityTable');
-    const expectedTitle = 'The title';
+    const expectedTitle = 'Course: The title';
 
     assert.equal(titleElement.text(), expectedTitle);
     assert.equal(tableElement.prop('title'), expectedTitle);