Fixes to maintenance stuff

farhatahmad · Mar 11, 2024 · 88a1a3b · 88a1a3b
1 parent 35a7227
commit 88a1a3b
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 8 deletions.
diff --git a/app/assets/locales/en.json b/app/assets/locales/en.json
@@ -413,7 +413,8 @@
         "brand_image_deleted": "The brand image has been deleted.",
         "privacy_policy_updated": "The privacy policy has been updated.",
         "helpcenter_updated": "The help center link has been updated.",
-        "terms_of_service_updated": "The terms of service have been updated."
+        "terms_of_service_updated": "The terms of service have been updated.",
+        "maintenance_updated": "The maintenance banner has been updated."
       },
       "recording": {
         "recording_visibility_updated": "The recording visibility has been updated.",

diff --git a/app/javascript/App.jsx b/app/javascript/App.jsx
@@ -50,7 +50,6 @@ export default function App() {
         progress: undefined,
         theme: 'light',
         className: 'text-center',
-        onClose: () => localStorage.setItem('maintenanceClosedAt', new Date().toISOString()),
       });
       localStorage.setItem('maintenanceBannerId', toastId);
     }

diff --git a/app/javascript/components/admin/site_settings/administration/LinksForm.jsx b/app/javascript/components/admin/site_settings/administration/LinksForm.jsx
@@ -27,7 +27,7 @@ export default function LinksForm({ id, value, mutation: useUpdateSiteSettingsAP
   const updateSiteSettingsAPI = useUpdateSiteSettingsAPI();
   const { t } = useTranslation();
 
-  const { methods, fields } = useLinksForm({ defaultValues: { value } });
+  const { methods, fields } = useLinksForm({ defaultValues: { value: value } });
 
   return (
     <Form id={id} methods={methods} onSubmit={updateSiteSettingsAPI.mutate}>
@@ -48,5 +48,5 @@ export default function LinksForm({ id, value, mutation: useUpdateSiteSettingsAP
 LinksForm.propTypes = {
   id: PropTypes.string.isRequired,
   mutation: PropTypes.func.isRequired,
-  value: PropTypes.string.isRequired,
+  value: PropTypes.string,
 };
diff --git a/app/javascript/components/admin/site_settings/administration/TextForm.jsx b/app/javascript/components/admin/site_settings/administration/TextForm.jsx
@@ -29,12 +29,13 @@ export default function TextForm({ id, value, mutation: useUpdateSiteSettingsAPI
   const { t } = useTranslation();
   const maintenanceBannerId = localStorage.getItem('maintenanceBannerId');
 
-  const { methods, fields } = useTextForm({ defaultValues: { value } });
+  const { methods, fields } = useTextForm({ defaultValues: { value: value } });
 
   // Function to clear the form
   const clearForm = () => {
-    methods.reset({ value: '' });
-    toast.dismiss(maintenanceBannerId);
+      methods.reset({ value: '' });
+      toast.dismiss(maintenanceBannerId);
+      updateSiteSettingsAPI.mutate('')
   };
 
   return (
@@ -60,5 +61,5 @@ export default function TextForm({ id, value, mutation: useUpdateSiteSettingsAPI
 TextForm.propTypes = {
   id: PropTypes.string.isRequired,
   mutation: PropTypes.func.isRequired,
-  value: PropTypes.string.isRequired,
+  value: PropTypes.string,
 };
diff --git a/app/javascript/hooks/mutations/admin/site_settings/useUpdateSiteSetting.jsx b/app/javascript/hooks/mutations/admin/site_settings/useUpdateSiteSetting.jsx
@@ -60,6 +60,9 @@ export default function useUpdateSiteSetting(name) {
       case 'TermsOfService':
         toast.success(t('toast.success.site_settings.terms_of_service_updated'));
         break;
+      case 'Maintenance':
+        toast.success(t('toast.success.site_settings.maintenance_updated'));
+        break;
       default:
         toast.success(t('toast.success.site_settings.site_setting_updated'));
     }