💄(domains) improve user experience and avoid repeat fix operations

Adds a loader after clicking on "Re-run check" for a domain

CHANGELOG.md

@@ -14,6 +14,7 @@ and this project adheres to
 
 ### Fixed
 
+- 💄(domains) improve user experience and avoid repeat fix_domain operations
 - 👽️(dimail) increase timeout value for check domain API call
 - 🧱(helm) add resource-server ingress path #743
 - 🌐(backend) synchronize translations with crowdin again

src/frontend/apps/desk/src/features/mail-domains/domains/components/MailDomainView.tsx

@@ -1,5 +1,6 @@
 import {
   Button,
+  Loader,
   Modal,
   ModalSize,
   VariantType,
@@ -62,18 +63,17 @@ export const MailDomainView = ({ mailDomain, onMailDomainUpdate }: Props) => {
     toast(t('copy done'), VariantType.SUCCESS);
   };
 
-  const { mutate: fetchMailDomain } = useFetchFromDimail({
+  const { mutate: fetchMailDomain, isPending: fetchPending } =
-    onSuccess: (data: MailDomain) => {
+    useFetchFromDimail({
-      console.info('fetchMailDomain success', data);
+      onSuccess: (data: MailDomain) => {
-      setShowModal(false);
+        setShowModal(false);
-      toast(t('Domain data fetched successfully'), VariantType.SUCCESS);
+        toast(t('Domain data fetched successfully'), VariantType.SUCCESS);
-      onMailDomainUpdate?.(data);
+        onMailDomainUpdate?.(data);
-    },
+      },
-    onError: () => {
+      onError: () => {
-      console.error('fetchMailDomain error');
+        toast(t('Failed to fetch domain data'), VariantType.ERROR);
-      toast(t('Failed to fetch domain data'), VariantType.ERROR);
+      },
-    },
+    });
-  });
 
   return (
     <>
@@ -155,13 +155,17 @@ export const MailDomainView = ({ mailDomain, onMailDomainUpdate }: Props) => {
           )}
           <pre>
             <div style={{ display: 'flex', justifyContent: 'center' }}>
-              <Button
+              {fetchPending ? (
-                onClick={() => {
+                <Loader />
-                  void fetchMailDomain(mailDomain.slug);
+              ) : (
-                }}
+                <Button
-              >
+                  onClick={() => {
-                {t('Re-run check')}
+                    void fetchMailDomain(mailDomain.slug);
-              </Button>
+                  }}
+                >
+                  {t('Re-run check')}
+                </Button>
+              )}
             </div>
           </pre>
         </Modal>