From 0984fb5b80712a32f5d0a65f53aaf9f379692474 Mon Sep 17 00:00:00 2001
From: Steven Lageveen <stevenmax888@gmail.com>
Date: Mon, 18 Nov 2024 13:40:50 +0100
Subject: [PATCH] Added tile grades for users with partial consent (0 goal
 grade) and better display this in student overview. Also added debug logging
 for assignment submissions error.

---
 .../src/components/pages/admin/studentoverview/settings.tsx | 6 +++---
 IguideME.Web/Services/LMSHandlers/CanvasHandler.cs          | 1 +
 IguideME.Web/Services/Workers/PeerGroupWorker.cs            | 2 +-
 charts/iguideme/Chart.yaml                                  | 4 ++--
 4 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/IguideME.Web/Frontend/src/components/pages/admin/studentoverview/settings.tsx b/IguideME.Web/Frontend/src/components/pages/admin/studentoverview/settings.tsx
index 086a9b6a..5987ae1a 100644
--- a/IguideME.Web/Frontend/src/components/pages/admin/studentoverview/settings.tsx
+++ b/IguideME.Web/Frontend/src/components/pages/admin/studentoverview/settings.tsx
@@ -99,11 +99,11 @@ export const getSettingsColumns = (): TableColumnsType<CommonData & SettingsData
             const consentB = b.consent === undefined ? 0 : +b.consent;
             return consentB - consentA;
           },
-          render: (value: ConsentEnum) => {
+          render: (value: ConsentEnum, record) => {
             if (value === ConsentEnum.Accepted) {
               return (
-                <Tooltip title='Consent given'>
-                  <CheckCircleOutlined className='text-success' />
+                <Tooltip title={record.goal ? 'Consent given' : 'Consent given without setting goal grade'}>
+                  <CheckCircleOutlined className={record.goal ? 'text-success' : 'text-meh'} />
                 </Tooltip>
               );
             } else if (value === ConsentEnum.Refused) {
diff --git a/IguideME.Web/Services/LMSHandlers/CanvasHandler.cs b/IguideME.Web/Services/LMSHandlers/CanvasHandler.cs
index 26062186..98288cae 100644
--- a/IguideME.Web/Services/LMSHandlers/CanvasHandler.cs
+++ b/IguideME.Web/Services/LMSHandlers/CanvasHandler.cs
@@ -185,6 +185,7 @@ private AppGradingType mapGradingType(GradingType type)
         /// <inheritdoc />
         public IEnumerable<AssignmentSubmission> GetSubmissions(int courseID, List<User> users)
         {
+            _logger.LogInformation("Getting submissions for users:\n{}", users.Select(user => user.UserID));
             return Connector
                 .FindCourseById(courseID)
                 .GetSubmissions(users.Select(user => user.UserID).ToArray(), false)
diff --git a/IguideME.Web/Services/Workers/PeerGroupWorker.cs b/IguideME.Web/Services/Workers/PeerGroupWorker.cs
index 2423f3df..64fdf0ec 100644
--- a/IguideME.Web/Services/Workers/PeerGroupWorker.cs
+++ b/IguideME.Web/Services/Workers/PeerGroupWorker.cs
@@ -66,7 +66,7 @@ List<string>[] groupUsers()
         {
             List<string>[] groupedUsers = new List<string>[11];
 
-            for (int goalGrade = 1; goalGrade <= 10; goalGrade++)
+            for (int goalGrade = 0; goalGrade <= 10; goalGrade++)
             {
                 groupedUsers[goalGrade] = _databaseManager.GetUserIDsWithGoalGrade(
                     this._courseID,
diff --git a/charts/iguideme/Chart.yaml b/charts/iguideme/Chart.yaml
index dcc5833a..db4a3f2c 100644
--- a/charts/iguideme/Chart.yaml
+++ b/charts/iguideme/Chart.yaml
@@ -2,5 +2,5 @@ apiVersion: v2
 name: iguideme
 description: IguideME
 type: application
-version: 0.3.238
-appVersion: "0.3.238"
+version: 0.3.239
+appVersion: "0.3.239"