fix: close modals only after mutation succeeds

Previously, modals closed immediately after firing mutations via
mutate(), giving no feedback on failure. Now the close action is
passed as an onSuccess callback so the modal stays open if the
mutation fails.

Co-Authored-By: Claude <noreply@anthropic.com>

Author: arbrandes

src/containers/EmailSettingsModal/hooks.js

@@ -21,8 +21,10 @@ export const useEmailData = ({
   const { mutate: updateEmailSettings } = useUpdateEmailSettings();
   const onToggle = () => setIsOptedOut(!isOptedOut);
   const save = () => {
-    updateEmailSettings({ courseId, enable: !isOptedOut });
-    closeModal();
+    updateEmailSettings(
+      { courseId, enable: !isOptedOut },
+      { onSuccess: () => closeModal() },
+    );
   };
 
   return {

src/containers/SelectSessionModal/hooks.js

@@ -40,10 +40,7 @@ export const useSelectSessionModalData = () => {
     isRefundable: courseData?.entitlement?.isRefundable || false,
   }), [courseData]);
   const { mutate: leaveEntitlement } = useDeleteEntitlementEnrollment();
-  const leaveEntitlementSession = () => leaveEntitlement({ uuid, isRefundable });
   const { mutate: switchEntitlementMutation } = useUpdateEntitlementEnrollment();
-  const switchEntitlementEnrollment = (selectedId) => switchEntitlementMutation({ uuid, courseId: selectedId });
-  const newEntitlementEnrollment = (selectedId) => switchEntitlementMutation({ uuid, courseId: selectedId });
 
   const [selectedSession, setSelectedSession] = module.state.selectedSession(courseId || null);
 
@@ -63,17 +60,17 @@ export const useSelectSessionModalData = () => {
 
   const handleSelection = ({ target: { value } }) => setSelectedSession(value);
   const handleSubmit = () => {
+    const onSuccess = () => closeSessionModal();
     if (selectedSession === LEAVE_OPTION) {
       trackLeaveSession();
-      leaveEntitlementSession();
+      leaveEntitlement({ uuid, isRefundable }, { onSuccess });
     } else if (isEnrolled) {
       trackSwitchSession();
-      switchEntitlementEnrollment(selectedSession);
+      switchEntitlementMutation({ uuid, courseId: selectedSession }, { onSuccess });
     } else {
       trackNewSession();
-      newEntitlementEnrollment(selectedSession);
+      switchEntitlementMutation({ uuid, courseId: selectedSession }, { onSuccess });
     }
-    closeSessionModal();
   };
 
   return {