fix form loading previous submission

Author: OliverW10

backend/GroupAllocator/Controllers/StudentsController.cs

@@ -4,6 +4,7 @@
 using GroupAllocator.Services;
 using Microsoft.AspNetCore.Authorization;
 using Microsoft.AspNetCore.Mvc;
+using Microsoft.AspNetCore.Mvc.ModelBinding;
 using Microsoft.EntityFrameworkCore;
 using Microsoft.IdentityModel.JsonWebTokens;
 
@@ -85,13 +86,14 @@ async Task<List<StudentInfoAndSubmission>> GetStudents(int classId)
 
 	[HttpGet("me")]
 	[Authorize(Policy = "StudentOnly")]
-	public async Task<ActionResult<StudentSubmissionDto>> Get(int classId)
+	public async Task<ActionResult<StudentSubmissionDto>> Get([BindRequired] int classId)
 	{
 		var userId = int.Parse(User.FindFirst(JwtRegisteredClaimNames.Sub)?.Value ?? throw new InvalidOperationException("No subject claim"));
-		
+		Console.WriteLine($"userId: {userId}, classId: {classId}");
 		var student = await db.Students
 			.Include(s => s.User)
 			.Include(s => s.Files)
+			.Include(s => s.Class)
 			.Include(s => s!.Preferences)
 				.ThenInclude(p => p.Project)
 			.Where(s => s.User.Id == userId && s.Class.Id == classId)

frontend/package.json

@@ -11,6 +11,7 @@
 	},
 	"dependencies": {
 		"@primeuix/themes": "^1.0.0",
+		"@primevue/core": "^4.3.7",
 		"@stripe/stripe-js": "^7.3.1",
 		"@unocss/reset": "66.1.0-beta.3",
 		"moment": "^2.30.1",

frontend/src/views/student/FormView.vue

@@ -113,6 +113,28 @@
 			</template>
 		</Card>
 		<LogoutButton />
+		
+		<!-- Success Dialog -->
+		<Dialog 
+			v-model:visible="showSuccessDialog" 
+			modal 
+			header="Success"
+			:closable="true"
+			:close-on-escape="true"
+			:style="{ width: '90vw', maxWidth: '500px' }"
+		>
+			<p class="text-base mb-2">Your project preferences have been successfully submitted.</p>
+			<p class="text-base mb-4">You can return to this page anytime to update your preferences.</p>
+			
+			<div class="flex flex-col sm:flex-row gap-2 justify-center">
+				<Button 
+					label="Close" 
+					@click="showSuccessDialog = false"
+					class="flex-1 sm:flex-none"
+					severity="secondary"
+				/>
+			</div>
+		</Dialog>
 	</div>
 </template>
 
@@ -135,6 +157,7 @@ import type { FileDetailsDto } from "../../dtos/file-details-dto";
 import type { StudentSubmissionDto } from "../../dtos/student-submission-dto";
 import { Column, DataTable } from "primevue";
 import { useRoute } from 'vue-router';
+import Dialog from "primevue/dialog";
 
 const toast = useToast();
 const route = useRoute();
@@ -147,6 +170,7 @@ const maxNumberOfPreferences = 10;
 const warningMessage = "A maximum of " + maxNumberOfPreferences + " preferences has been selected. Anything more than the top " + maxNumberOfPreferences + " preferences will not be saved";
 let exceededPreferenceLimit = false
 const loading = ref(false)
+const showSuccessDialog = ref(false)
 
 const projectsRaw = ref([] as ProjectDto[]);
 const projects = ref([[], []] as ProjectDto[][]);
@@ -166,7 +190,7 @@ onMounted(async () => {
 	loading.value = true
 	try{
 		await loadProjects()
-		const maybeStudent = await ApiService.get<StudentSubmissionDto | undefined>("/students/me")
+		const maybeStudent = await ApiService.get<StudentSubmissionDto | undefined>(`/students/me?classId=${classId}`)
 		if (maybeStudent) {
 			console.log(maybeStudent)
 			files.value = maybeStudent.files ?? []
@@ -241,7 +265,7 @@ const submitForm = async () => {
 	if (result == null) {
 		toast.add({ severity: 'error', summary: 'Failed', detail: 'Submission failed. If the issue persists contact developers', life: 5000 });
 	} else {
-		toast.add({ severity: 'success', summary: 'Success', detail: 'Submitted preferences', life: 5000 });
+		showSuccessDialog.value = true;
 	}
 };
 

frontend/src/views/teacher/TeachersAdminPage.vue

@@ -92,6 +92,7 @@ const addTeacher = async () => {
 
 		teachers.value.push({ email: newTeacherEmail.value.trim(), isOwner: false })
 		newTeacherEmail.value = ''
+		toast.add({ severity: 'success', summary: 'Success', detail: `Teacher ${newTeacherEmail.value.trim()} added successfully`, life: 3000 });
 	} catch {
 		toast.add({ severity: 'error', summary: 'Failed', detail: `Failed to add teacher`, life: 3000 });
 	} finally {
@@ -107,6 +108,7 @@ const deleteTeacher = async (teacherEmail: string) => {
 	try {
 		await ApiService.delete(`/class/${classId}/remove-teacher/${encodeURIComponent(teacherEmail)}`)
 		teachers.value = teachers.value.filter(t => t.email !== teacherEmail)
+		toast.add({ severity: 'success', summary: 'Success', detail: `Teacher ${teacherEmail} removed successfully`, life: 3000 });
 	} catch {
 		toast.add({ severity: 'error', summary: 'Failed', detail: `Failed to remove teacher ${teacherEmail}`, life: 3000 });
 	} finally {