From 5f367a95405db116fda8c8b0ecad1c446650c331 Mon Sep 17 00:00:00 2001
From: Rotimi Best <best@classroomio.com>
Date: Tue, 31 Dec 2024 17:47:07 +0100
Subject: [PATCH] chore: sort top courses by completion and spacing fixes #531

---
 apps/dashboard/src/routes/org/[slug]/+page.svelte      | 10 ++++++----
 supabase/migrations/20241231154543_dashboard_stats.sql |  1 +
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/apps/dashboard/src/routes/org/[slug]/+page.svelte b/apps/dashboard/src/routes/org/[slug]/+page.svelte
index 98285211..1d6f53cf 100644
--- a/apps/dashboard/src/routes/org/[slug]/+page.svelte
+++ b/apps/dashboard/src/routes/org/[slug]/+page.svelte
@@ -150,10 +150,12 @@
           {/each}
         {:else}
           {#each dashAnalytics.topCourses as course}
-            <div class="flex items-center">
-              <div class="space-y-1">
+            <div class="flex items-center gap-2">
+              <div class="w-4/6 space-y-1">
                 <Link href={`/courses/${course.id}`}>
-                  <p class="text-sm font-medium leading-none">{course.title}</p>
+                  <p class="line-clamp-2 pb-[0.1rem] text-sm font-medium leading-none">
+                    {course.title}
+                  </p>
                 </Link>
                 <p class="text-muted-foreground text-sm">
                   {course.enrollments}
@@ -216,7 +218,7 @@
                 />
 
                 <div class="min-h-[45px] space-y-1">
-                  <p class="text-sm font-medium leading-none">{enrollment.name}</p>
+                  <p class="text-sm font-medium capitalize leading-none">{enrollment.name}</p>
                   <p class="text-muted-foreground text-sm">
                     <span class="italic">
                       {calDateDiff(enrollment.date)}
diff --git a/supabase/migrations/20241231154543_dashboard_stats.sql b/supabase/migrations/20241231154543_dashboard_stats.sql
index 95f89783..87127538 100644
--- a/supabase/migrations/20241231154543_dashboard_stats.sql
+++ b/supabase/migrations/20241231154543_dashboard_stats.sql
@@ -72,6 +72,7 @@ SELECT
       AND lc.profile_id = gm.profile_id
     WHERE c.status = 'ACTIVE'  AND g.organization_id = org_id_arg
     GROUP BY c.id, c.title
+    ORDER BY completed_lessons DESC
   ) as course_stats
   LIMIT 5;
 END;