fix(user): prompt before sending lab rankings

BastiDood · Jul 11, 2024 · 950322f · 950322f
1 parent 64ba4f3
commit 950322f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/src/routes/dashboard/(draft)/ranks/+page.server.js b/src/routes/dashboard/(draft)/ranks/+page.server.js
@@ -23,7 +23,7 @@ export const actions = {
 
         const maxRounds = await db.getMaxRoundInDraft(draft);
         if (maxRounds === null) error(404);
-        if (labs.length !== maxRounds) error(400);
+        if (labs.length > maxRounds) error(400);
 
         if (await db.insertStudentRanking(draft, user.email, labs)) return;
         return fail(403);

diff --git a/src/routes/dashboard/(draft)/ranks/SubmitRankings.svelte b/src/routes/dashboard/(draft)/ranks/SubmitRankings.svelte
@@ -70,7 +70,11 @@
 <form
     method="post"
     class="space-y-4"
-    use:enhance={({ submitter }) => {
+    use:enhance={({ submitter, cancel }) => {
+        if (!confirm(`Are you sure you want to select ${selectedLabs.length} labs?`)) {
+            cancel();
+            return;
+        }
         assert(submitter !== null);
         assert(submitter instanceof HTMLButtonElement);
         submitter.disabled = true;
@@ -106,9 +110,7 @@
                 You may no longer select any more labs.
             {/if}
         </p>
-        <button type="submit" disabled={hasRemaining} class="variant-filled-primary btn disabled:!variant-ghost-primary"
-            >Submit Lab Preferences</button
-        >
+        <button type="submit" class="variant-filled-primary btn">Submit Lab Preferences</button>
     </div>
     <hr class="!border-surface-400-500-token !border-t-4" />
     {#each selectedLabs as { lab_id, lab_name }, idx (lab_id)}