fix: use better type assertion for initialAsyncRequestState (#42087) (#42137)

(cherry picked from commit ca7a62682e)

Co-authored-by: Gilles De Mey <gilles.de.mey@gmail.com>

public/app/features/alerting/unified/AmRoutes.tsx

@@ -58,6 +58,10 @@ const AmRoutes: FC = () => {
 
   useCleanup((state) => state.unifiedAlerting.saveAMConfig);
   const handleSave = (data: Partial<FormAmRoute>) => {
+    if (!result) {
+      return;
+    }
+
     const newData = formAmRouteToAmRoute(
       alertManagerSourceName,
       {

public/app/features/alerting/unified/components/admin/AlertmanagerConfig.tsx

@@ -55,7 +55,7 @@ export default function AlertmanagerConfig(): JSX.Element {
   const loading = isDeleting || isLoadingConfig || isSaving;
 
   const onSubmit = (values: FormValues) => {
-    if (alertManagerSourceName) {
+    if (alertManagerSourceName && config) {
       dispatch(
         updateAlertManagerConfigAction({
           newConfig: JSON.parse(values.configJSON),

public/app/features/alerting/unified/utils/redux.ts

@@ -13,8 +13,13 @@ export interface AsyncRequestState<T> {
   requestId?: string;
 }
 
-export const initialAsyncRequestState: AsyncRequestState<any> = Object.freeze({
+export const initialAsyncRequestState: Pick<
+  AsyncRequestState<undefined>,
+  'loading' | 'dispatched' | 'result' | 'error'
+> = Object.freeze({
   loading: false,
+  result: undefined,
+  error: undefined,
   dispatched: false,
 });