Fix notifications settings that were not remembered

src/app/Pages/ActiveDeals/ActiveDeals.scss

@@ -63,7 +63,7 @@
     justify-content: flex-start;
     padding-bottom: 4px;
 
-    .syncToggles{
+    .notificationsSettings {
         flex: 1;
         align-self: flex-start;
         display: flex;
@@ -73,7 +73,7 @@
     }
 
 
-    .tableButtons{
+    .tableButtons {
         flex-basis: 20%;
         align-items: center;
         display: flex;

src/app/Pages/ActiveDeals/ActiveDeals.tsx

@@ -5,7 +5,7 @@ import { formatDeals } from '@/app/Components/DataTable/Index'
 
 import { useAppSelector } from '@/app/redux/hooks';
 import { Card_ActiveDeals, Card_totalInDeals, Card_ActiveDealReserve, Card_TotalDayProfit, Card_TotalUnrealizedProfit, Card_TotalRoi } from '@/app/Components/Charts/DataCards';
-import { SyncToggles } from './Components/index';
+import { NotificationsSettings } from './Components/index';
 
 import './ActiveDeals.scss'
 
@@ -51,7 +51,7 @@ const ActiveDealsPage = () => {
             <div className="boxData flex-column" style={{padding: '.5em 1em 1em', overflow: 'hidden'}}>
                 <div className="tableSettings">
 
-                    <SyncToggles />
+                    <NotificationsSettings />
 
                     <div className="filters tableButtons" >
                     <ToggleRefreshButton style={{ width: '250px', margin: '5px', height: '38px' }} className={"ToggleRefreshButton"}  />

src/app/Pages/ActiveDeals/Components/NotificationsSettings.tsx

@@ -0,0 +1,63 @@
+import { useAppSelector } from '@/app/redux/hooks';
+import { Checkbox, FormControlLabel } from '@mui/material';
+import React, { useState } from "react";
+import { updateNotificationsSettingsGlobal } from '../../../redux/configActions';
+
+/**
+ * 
+ * @returns Checkboxes for configuring the state of auto sync.
+ */
+const NotificationsSettings = () => {
+
+    const { config } = useAppSelector(state => state.config);
+
+    const [summary, setSummary] = useState(() => config.globalSettings.notifications.summary)
+    const [enabled, setEnabled] = useState(() => config.globalSettings.notifications.enabled)
+
+    const changeSummary = (event: React.ChangeEvent<HTMLInputElement>) => {
+        const checked = event.target.checked
+        updateNotificationsSettingsGlobal({ summary: checked })
+        setSummary(checked)
+    }
+
+    const changeEnabled = (event: React.ChangeEvent<HTMLInputElement>) => {
+        const checked = event.target.checked
+        updateNotificationsSettingsGlobal({ enabled: checked })
+        setEnabled(checked)
+    }
+
+    return (
+        <div className="notificationsSettings">
+            <FormControlLabel
+                control={
+                    <Checkbox
+                        checked={enabled}
+                        onChange={changeEnabled}
+                        name="notifications"
+                        color="primary"
+                        style={{ color: 'var(--color-secondary)' }}
+
+                    />
+                }
+                label="Enable Notifications"
+
+            />
+            <FormControlLabel
+                control={
+                    <Checkbox
+                        checked={summary}
+                        onChange={changeSummary}
+                        name="summary"
+                        style={{ color: 'var(--color-secondary)' }}
+
+                    />
+                }
+                label="Summarize Notifications"
+            />
+
+
+        </div>
+    )
+}
+
+export default NotificationsSettings;

src/app/Pages/ActiveDeals/Components/index.ts

@@ -1,8 +1,8 @@
 
-import SyncToggles from "./SyncToggles";
+import NotificationsSettings from "./NotificationsSettings";
 import SubRowAsync from "./Subrow";
 
 export {
-    SyncToggles,
+    NotificationsSettings,
     SubRowAsync
 }

src/app/redux/configActions.ts

@@ -1,7 +1,7 @@
-import { setConfig, setCurrentProfile, updateCurrentProfileByPath, deleteProfileById } from '@/app/redux/configSlice'
+import { setConfig, setCurrentProfile, updateCurrentProfileByPath, deleteProfileById, updateNotificationsSettings } from '@/app/redux/configSlice'
 import {setSyncData} from '@/app/redux/threeCommas/threeCommasSlice'
 
-import { TconfigValues, Type_Profile, Type_ReservedFunds } from '@/types/config';
+import { TconfigValues, Type_NotificationsSettings, Type_Profile, Type_ReservedFunds } from '@/types/config';
 
 import { removeDuplicatesInArray } from '@/utils/helperFunctions';
 
@@ -127,12 +127,16 @@ const deleteProfileByIdGlobal = (config: TconfigValues, profileId:string, setOpe
     }
 }
 
-
+const updateNotificationsSettingsGlobal = async (settings: Partial<Type_NotificationsSettings>) => {
+    store.dispatch(updateNotificationsSettings(settings))
+    await storeConfigInFile()
+}
 
 export {
     updateConfig,
     updateReservedFundsArray,
     updateNestedCurrentProfile,
     storeConfigInFile,
-    deleteProfileByIdGlobal
+    deleteProfileByIdGlobal,
+    updateNotificationsSettingsGlobal,
 }

src/app/redux/configSlice.ts

@@ -1,9 +1,8 @@
-import { createSlice } from '@reduxjs/toolkit'
-import type { RootState } from './store'
+import { TconfigValues, Type_Profile, Type_ReservedFunds } from '@/types/config';
+import { defaultConfig, defaultProfile } from '@/utils/defaultConfig';
+import { createSlice } from '@reduxjs/toolkit';
 import { v4 as uuidv4 } from 'uuid';
 
-import { Type_Profile, TconfigValues, Type_ReservedFunds } from '@/types/config'
-import { defaultProfile, defaultConfig } from '@/utils/defaultConfig'
 
 
 const defaultReservedFunds = {
@@ -18,7 +17,6 @@ const defaultReservedFunds = {
 const initialState = {
     config: <TconfigValues>defaultConfig,
     currentProfile: <Type_Profile>defaultProfile,
-    reservedFunds: <Type_ReservedFunds[]>[defaultReservedFunds]
 }
 
 const configPaths = {
@@ -122,15 +120,24 @@ export const configSlice = createSlice({
         },
         addConfigProfile: state => {
             state.currentProfile = { ...defaultProfile, id: uuidv4() }
-        }
+        },
+        updateNotificationsSettings: (state, action) => {
+            const newConfig = { ...state.config }
+            newConfig.globalSettings.notifications = {
+                ...state.config.globalSettings.notifications,
+                ...action.payload,
+            }
+            state.config = newConfig
+        },
     }
 })
 
 export const {
     setConfig, setCurrentProfile, 
     updateCurrentProfileByPath, deleteProfileById, addConfigProfile,
     setCurrentProfileById,
-    updateLastSyncTime
+    updateLastSyncTime,
+    updateNotificationsSettings,
 } = configSlice.actions;
 export { configPaths }
 

src/app/redux/threeCommas/Actions.ts

@@ -223,13 +223,15 @@ const updateAllData = async (offset: number = 1000, profileData: Type_Profile, t
     const syncOptions = store.getState().threeCommas.syncOptions
     store.dispatch(setIsSyncing(true))
 
-    let time = (syncOptions.time) ? syncOptions.time : 0;
-    let syncCount = (syncOptions.syncCount) ? syncOptions.syncCount : 0;
-    if(type === 'fullSync') syncCount = 0;
+    const originalTime = syncOptions.time || 0
+    let time = originalTime
+    let syncCount = syncOptions.syncCount || 0
+    if (type === 'fullSync') {
+        syncCount = 0
+        time = 0
+    }
     const options = {
         syncCount,
-        summary: (syncOptions.summary) ? syncOptions.summary : false,
-        notifications: (syncOptions.notifications) ? syncOptions.notifications : false,
         time,
         offset
     }
@@ -244,10 +246,11 @@ const updateAllData = async (offset: number = 1000, profileData: Type_Profile, t
             .then((lastSyncTime) => {
                 store.dispatch(setSyncData({
                     syncCount:(type === 'autoSync') ? options.syncCount + 1 : 0,
-                    time: (type === 'autoSync') ? options.time + 15000 : 0
+                    // don't override syncOptions.time in case of a fullSync
+                    // because there might be a concurrent autoSync running
+                    time: (type === 'autoSync') ? originalTime + 15000 : originalTime
                 }))
                 store.dispatch(updateLastSyncTime({data: lastSyncTime }))
-
             })
 
     } catch (error) {

src/app/redux/threeCommas/initialState.ts

@@ -61,8 +61,6 @@ export const initialState = {
 export type typeString = 'botData' | 'profitData' | 'activeDeals' | 'performanceData' | 'metricsData' | 'accountData' | 'balanceData'
 
 export type Type_SyncData = {
-    summary?: boolean,
-    notifications?: boolean,
     time?: number,
     syncCount?: number
 }

src/main/3Commas/index.ts

@@ -1,4 +1,6 @@
 import { update, run, query } from '@/main/Database/database';
+import {config} from '@/main/Config/config';
+
 const { bots, getAccountDetail, deals, getAccountSummary, getDealOrders } = require('./api');
 const log = require('electron-log');
 
@@ -37,8 +39,12 @@ async function getDealData(type: string, options: Type_UpdateFunction, profileDa
 
       let { deals, lastSyncTime } = data
 
+      const enabled = config.get('globalSettings.notifications.enabled')
+      const summary = config.get('globalSettings.notifications.summary')
       // if notifications need to be enabled for the fullSync then the type below needs to be updated.
-      if (type === 'autoSync' && options.notifications && options.time != undefined || options.syncCount != 0) findAndNotifyNewDeals(deals, options.time, options.summary)
+      if (type === 'autoSync' && enabled && options.time != undefined || options.syncCount != 0) {
+        findAndNotifyNewDeals(deals, options.time, summary)
+      }
       update('deals', deals, profileData.id)
       // log.info(data)
 

src/types/3Commas.ts

@@ -274,8 +274,6 @@ export type Type_MarketOrders = {
 export interface Type_UpdateFunction{
     offset: number
     time:number
-    summary: boolean
-    notifications: boolean
     syncCount: number
 }
 

src/types/config.ts

@@ -1,8 +1,9 @@
-import type { supportedCurrencies} from'@/utils/granularity'
+import type { supportedCurrencies } from '@/utils/granularity'
 
 export interface TconfigValues {
     profiles: Record<string, Type_Profile>,
     current: string,
+    globalSettings: Type_GlobalSettings,
     general: {
         version: string
     },
@@ -38,6 +39,16 @@ export interface Type_Profile {
 
 }
 
+export interface Type_GlobalSettings {
+    notifications: Type_NotificationsSettings
+}
+
+export interface Type_NotificationsSettings {
+    enabled: boolean,
+    summary: boolean,
+}
+
+
 export interface Type_ReservedFunds {
     id: number
     account_name: string
@@ -64,7 +75,7 @@ export interface Type_ConfigContext {
         currency: string[]
         updateCurrency: any
         updateApiData: any
-        apiData: {key: string, secret: string, mode: string}
+        apiData: { key: string, secret: string, mode: string }
         reservedFunds: Type_ReservedFunds[],
         updateReservedFunds: any
         currentProfileId: string

src/utils/defaultConfig.ts

@@ -101,9 +101,15 @@ const defaultProfile:Type_Profile = {
 }
 
 
-const defaultConfig = {
-    profiles: Object.assign({[currentId]: defaultProfile}), 
+const defaultConfig: TconfigValues = {
+    profiles: Object.assign({[currentId]: defaultProfile}),
     current: currentId,
+    globalSettings: {
+        notifications: {
+            enabled: true,
+            summary: false,
+        },
+    },
     general: {
         version : version
     },