fix: Delete async await

Change updateFilterAndFetchDataset function
Delete the try catch in the outer layer of request2

Author: SoarBlueSky

frontend/src/app/components/ChartEditor.tsx

@@ -296,7 +296,7 @@ export const ChartEditor: FC<ChartEditorProps> = ({
       chartConfig?.datas,
       drillOptionRef.current,
     );
-  }, [dispatch, chart?.meta?.id, registerChartEvents]);
+  }, [dispatch, chart?.meta?.id, registerChartEvents, chartConfig?.datas]);
 
   const handleChartChange = (c: IChart) => {
     registerChartEvents(c);
@@ -327,7 +327,7 @@ export const ChartEditor: FC<ChartEditorProps> = ({
   };
 
   const handleChartConfigChange = useCallback(
-    async (type, payload) => {
+    (type, payload) => {
       if (expensiveQuery) {
         dispatch(
           workbenchSlice.actions.updateChartConfig({
@@ -357,7 +357,7 @@ export const ChartEditor: FC<ChartEditorProps> = ({
           });
         }
 
-        await dispatch(
+        dispatch(
           workbenchSlice.actions.updateCurrentDataViewComputedFields(
             computedFields,
           ),

frontend/src/app/pages/ChartWorkbenchPage/components/ChartOperationPanel/components/ChartDataViewPanel/ChartDataViewPanel.tsx

@@ -67,12 +67,6 @@ const ChartDataViewPanel: FC<{
   const [isDisplayAddNewModal, setIsDisplayAddNewModal] = useToggle();
   const views = useSelector(dataviewsSelector);
 
-  useMount(() => {
-    if (defaultViewId) {
-      handleDataViewChange(defaultViewId);
-    }
-  });
-
   const path = useMemo(() => {
     return views?.length && dataView
       ? getPath(

frontend/src/app/pages/MainPage/pages/ViewPage/slice/thunks.ts

@@ -332,17 +332,13 @@ export const deleteView = createAsyncThunk<
   DeleteViewParams,
   { state: RootState }
 >('view/deleteView', async ({ id, archive, resolve }, { dispatch }) => {
-  try {
-    await request2<boolean>({
-      url: `/views/${id}`,
-      method: 'DELETE',
-      params: { archive },
-    });
-    resolve();
-    return null;
-  } catch (error) {
-    throw error;
-  }
+  await request2<boolean>({
+    url: `/views/${id}`,
+    method: 'DELETE',
+    params: { archive },
+  });
+  resolve();
+  return null;
 });
 
 export const getEditorProvideCompletionItems = createAsyncThunk<

frontend/src/app/pages/MainPage/pages/VizPage/ChartPreview/ChartPreview.tsx

@@ -389,7 +389,7 @@ const ChartPreviewBoard: FC<{
       );
     };
 
-    const handleChartDrillDataAggregationChange = async (type, payload) => {
+    const handleChartDrillDataAggregationChange = (type, payload) => {
       const rows = getInterimDateAggregateRows(payload.value?.rows);
       const dateAggregationField = rows.filter(
         v => v.category === ChartDataViewFieldCategory.DateAggregationField,
@@ -402,13 +402,13 @@ const ChartPreviewBoard: FC<{
         chartPreview?.chartConfig,
       );
 
-      await dispatch(
+      dispatch(
         vizAction.updateComputedFields({
           backendChartId,
           computedFields,
         }),
       );
-      await dispatch(
+      dispatch(
         updateGroupAndFetchDataset({
           backendChartId,
           payload: payload,

frontend/src/app/pages/MainPage/pages/VizPage/slice/thunks.ts

@@ -326,7 +326,7 @@ export const updateFilterAndFetchDataset = createAsyncThunk(
     thunkAPI,
   ) => {
     await thunkAPI.dispatch(
-      vizActions.updateChartPreviewGroup({
+      vizActions.updateChartPreviewFilter({
         backendChartId: arg.backendChartId,
         payload: arg.payload,
       }),

frontend/src/app/pages/SharePage/ChartForShare.tsx

@@ -151,7 +151,8 @@ const ChartForShare: FC<{
       }),
     );
   };
-  const handleChartDrillDataAggregationChange = async (type, payload) => {
+
+  const handleChartDrillDataAggregationChange = (type, payload) => {
     const rows = getInterimDateAggregateRows(payload.value?.rows);
     const dateAggregationField = rows.filter(
       v => v.category === ChartDataViewFieldCategory.DateAggregationField,
@@ -164,20 +165,21 @@ const ChartForShare: FC<{
       chartPreview?.chartConfig,
     );
 
-    await dispatch(
+    dispatch(
       shareActions.updateComputedFields({
         backendChartId: chartPreview?.backendChart?.id!,
         computedFields,
       }),
     );
-    await dispatch(
+    dispatch(
       updateGroupAndFetchDatasetForShare({
         backendChartId: chartPreview?.backendChart?.id!,
         payload: payload,
         drillOption: drillOptionRef?.current,
       }),
     );
   };
+
   return (
     <StyledChartPreviewBoard>
       <div ref={controlRef}>