feat: lesson details and lesson progress

apps/api/src/chapter/chapter.service.ts

@@ -2,7 +2,7 @@ import { Inject, Injectable, NotFoundException, UnauthorizedException } from "@n
 import { EventBus } from "@nestjs/cqrs";
 
 import { DatabasePg } from "src/common";
-import { LessonRepository } from "src/lesson/lesson.repository";
+import { LessonRepository } from "src/lesson/repositories/lesson.repository";
 
 import { ChapterRepository } from "./repositories/chapter.repository";
 

apps/api/src/chapter/chapter.type.ts

@@ -1,8 +1,4 @@
-export const LESSON_TYPE = {
-  quiz: { key: "quiz", value: "Quiz" },
-  multimedia: { key: "multimedia", value: "Multimedia" },
-} as const;
-
+// TODO: remove unused types
 export const LESSON_ITEM_TYPE = {
   text_block: { key: "text_block", value: "Text Block" },
   file: { key: "file", value: "File" },

apps/api/src/courses/course.controller.ts

@@ -171,6 +171,7 @@ export class CourseController {
   })
   async getTeacherCourses(
     @Query("authorId") authorId: UUIDType,
+    // TODO: extract to const
     @Query("scope") scope: "all" | "enrolled" | "available" = "all",
     @Query("excludeCourseId") excludeCourseId: UUIDType,
     @CurrentUser("userId") currentUserId: UUIDType,

apps/api/src/courses/course.service.ts

@@ -20,7 +20,6 @@ import {
   sql,
 } from "drizzle-orm";
 
-import { LESSON_TYPE } from "src/chapter/chapter.type";
 import { AdminChapterRepository } from "src/chapter/repositories/adminChapter.repository";
 import { DatabasePg } from "src/common";
 import { addPagination, DEFAULT_PAGE_SIZE } from "src/common/pagination";
@@ -84,6 +83,7 @@ export class CourseService {
       currentUserRole,
     } = query;
 
+    // TODO: repair it
     // const { sortOrder, sortedField } = getSortOptions(sort);
     const sortOrder = asc;
     const sortedField = CourseSortFields.title;
@@ -261,15 +261,15 @@ export class CourseService {
           priceInCents: courses.priceInCents,
           currency: courses.currency,
           hasFreeChapters: sql<boolean>`
-        EXISTS (
-          SELECT 1
-          FROM ${chapters}
-          WHERE ${chapters.courseId} = ${courses.id}
-            AND ${chapters.isFreemium} = TRUE
-        )`,
+            EXISTS (
+              SELECT 1
+              FROM ${chapters}
+              WHERE ${chapters.courseId} = ${courses.id}
+                AND ${chapters.isFreemium} = TRUE
+            )
+          `,
         })
         .from(courses)
-        .leftJoin(studentCourses, eq(studentCourses.courseId, courses.id))
         .leftJoin(categories, eq(courses.categoryId, categories.id))
         .leftJoin(users, eq(courses.authorId, users.id))
         .leftJoin(coursesSummaryStats, eq(courses.id, coursesSummaryStats.courseId))
@@ -283,11 +283,9 @@ export class CourseService {
           users.firstName,
           users.lastName,
           users.email,
-          studentCourses.studentId,
           categories.title,
           coursesSummaryStats.freePurchasedCount,
           coursesSummaryStats.paidPurchasedCount,
-          studentCourses.finishedChapterCount,
         )
         .orderBy(sortOrder(this.getColumnToSortBy(sortedField as CourseSortField)));
 
@@ -297,7 +295,6 @@ export class CourseService {
       const [{ totalItems }] = await trx
         .select({ totalItems: countDistinct(courses.id) })
         .from(courses)
-        .leftJoin(studentCourses, eq(studentCourses.courseId, courses.id))
         .leftJoin(categories, eq(courses.categoryId, categories.id))
         .leftJoin(users, eq(courses.authorId, users.id))
         .where(and(...conditions));
@@ -369,7 +366,7 @@ export class CourseService {
           (SELECT COUNT(*)
           FROM ${lessons}
           WHERE ${lessons.chapterId} = ${chapters.id}
-            AND ${lessons.type} = ${LESSON_TYPE.quiz.key})::INTEGER`,
+            AND ${lessons.type} = ${LESSON_TYPES.QUIZ})::INTEGER`,
         completedLessonCount: sql<number>`COALESCE(${studentChapterProgress.completedLessonCount}, 0)`,
         chapterProgress: sql<ProgressStatus>`
           CASE
@@ -398,7 +395,7 @@ export class CourseService {
                     ELSE 'not_started'
                   END AS status,
                   CASE
-                    WHEN ${lessons.type} = ${LESSON_TYPES.quiz} THEN COUNT(${questions.id})
+                    WHEN ${lessons.type} = ${LESSON_TYPES.QUIZ} THEN COUNT(${questions.id})
                     ELSE NULL
                   END AS "quizQuestionCount"
                 FROM ${lessons}
@@ -590,9 +587,9 @@ export class CourseService {
         excludeCourseId,
       );
 
-      if (availableCourseIds.length) {
-        conditions.push(inArray(courses.id, availableCourseIds));
-      }
+      if (!availableCourseIds.length) return [];
+
+      conditions.push(inArray(courses.id, availableCourseIds));
     }
 
     return this.db
@@ -620,7 +617,10 @@ export class CourseService {
         )`,
       })
       .from(courses)
-      .leftJoin(studentCourses, eq(studentCourses.courseId, courses.id))
+      .leftJoin(
+        studentCourses,
+        and(eq(studentCourses.courseId, courses.id), eq(studentCourses.studentId, currentUserId)),
+      )
       .leftJoin(categories, eq(courses.categoryId, categories.id))
       .leftJoin(users, eq(courses.authorId, users.id))
       .where(and(...conditions))
@@ -823,7 +823,7 @@ export class CourseService {
               studentId,
               lessonId: lesson.id,
               completedQuestionCount: 0,
-              quizScore: lesson.type === LESSON_TYPES.quiz ? 0 : null,
+              quizScore: lesson.type === LESSON_TYPES.QUIZ ? 0 : null,
               completedAt: null,
             })),
           );
@@ -934,7 +934,7 @@ export class CourseService {
         const updatedLesson = { ...lesson };
         if (
           lesson.fileS3Key &&
-          (lesson.type === LESSON_TYPES.video || lesson.type === LESSON_TYPES.presentation)
+          (lesson.type === LESSON_TYPES.VIDEO || lesson.type === LESSON_TYPES.PRESENTATION)
         ) {
           if (!lesson.fileS3Key.startsWith("https://")) {
             try {