Add functionality to return student remixes with unread feedback count

Author: jamiebenstead

app/controllers/api/school_classes_controller.rb

@@ -14,7 +14,7 @@ def index
       if current_user&.school_teacher?(@school) || current_user&.school_owner?(@school)
         render :teacher_index, formats: [:json], status: :ok
       else
-        render :student_index, formats: [:json], status: :ok
+        render_student_index(school_classes)
       end
     end
 
@@ -83,6 +83,43 @@ def destroy
 
     private
 
+    def render_student_index(school_classes)
+      unread_counts = calculate_unread_counts(school_classes)
+      @school_classes_with_teachers_and_unread_counts = @school_classes_with_teachers.zip(unread_counts)
+      render :student_index, formats: [:json], status: :ok
+    end
+
+    def calculate_unread_counts(school_classes)
+      school_classes.map do |school_class|
+        lessons = school_class.lessons.accessible_by(current_ability)
+        remixes = user_remixes_for_lessons(lessons)
+
+        remixes.count do |remix|
+          remix&.school_project&.feedback&.exists?(read_at: nil)
+        end
+      end
+    end
+
+    def user_remixes_for_lessons(lessons)
+      lessons.filter_map do |lesson|
+        next nil unless lesson&.project&.remixes&.any?
+
+        user_remix_for_lesson(lesson)
+      end
+    end
+
+    def user_remix_for_lesson(lesson)
+      remixes = lesson.project&.remixes
+      return unless remixes
+
+      remixes
+        .where(user_id: current_user.id)
+        .accessible_by(current_ability)
+        .order(created_at: :asc)
+        .includes(school_project: :feedback)
+        .first
+    end
+
     def find_or_create_school_class(school_class_params)
       # First try and find the class (in case we're re-importing)
       existing_school_class = SchoolClass.find_by(

app/views/api/school_classes/student_index.json.jbuilder

@@ -1,5 +1,8 @@
 # frozen_string_literal: true
 
-json.array!(@school_classes_with_teachers) do |school_class, teachers|
+json.array!(@school_classes_with_teachers_and_unread_counts) do |class_with_teachers, unread_count|
+  school_class, teachers = class_with_teachers
+
   json.partial! 'school_class', school_class: school_class, teachers: teachers
+  json.unread_feedback_count unread_count
 end