fix(exams): add confirmation dialog when accepting or rejecting a pro…

…visional grade
polito · Feb 6, 2024 · 243d75f · 243d75f
1 parent 4501998
commit 243d75f
Showing 1 changed file with 14 additions and 10 deletions.
diff --git a/src/features/transcript/screens/ProvisionalGradeScreen.tsx b/src/features/transcript/screens/ProvisionalGradeScreen.tsx
@@ -130,11 +130,13 @@ export const ProvisionalGradeScreen = ({ navigation, route }: Props) => {
           <CtaButton
             title={t('provisionalGradeScreen.acceptGradeCta')}
             action={() =>
-              confirmAcceptance().then(() =>
-                acceptGradeQuery
-                  .mutateAsync(grade.id)
-                  .then(() => provideFeedback(true)),
-              )
+              confirmAcceptance().then(ok => {
+                if (ok) {
+                  acceptGradeQuery
+                    .mutateAsync(grade.id)
+                    .then(() => provideFeedback(true));
+                }
+              })
             }
             variant="outlined"
             absolute={false}
@@ -149,11 +151,13 @@ export const ProvisionalGradeScreen = ({ navigation, route }: Props) => {
           <CtaButton
             title={t('provisionalGradeScreen.rejectGradeCta')}
             action={() =>
-              confirmRejection().then(() =>
-                rejectGradeQuery
-                  .mutateAsync(grade.id)
-                  .then(() => provideFeedback(false)),
-              )
+              confirmRejection().then(ok => {
+                if (ok) {
+                  rejectGradeQuery
+                    .mutateAsync(grade.id)
+                    .then(() => provideFeedback(false));
+                }
+              })
             }
             absolute={false}
             loading={rejectGradeQuery.isLoading}