Fix linting errors, add TODOs for failing tx flows

src/components/safe-apps/SafeAppsSignMessageModal/ReviewSafeAppsSignMessage.tsx

@@ -92,8 +92,9 @@ const ReviewSafeAppsSignMessage = ({
     }
   }
 
+  // TODO: Need to use the SafeTxProvider here
   return (
-    <SignOrExecuteForm safeTx={safeTx} onSubmit={handleSubmit} error={safeTxError || submitError}>
+    <SignOrExecuteForm onSubmit={handleSubmit}>
       <>
         <SendFromBlock />
 

src/components/safe-apps/SafeAppsTxModal/ReviewSafeAppsTx.tsx

@@ -61,8 +61,9 @@ const ReviewSafeAppsTx = ({
 
   const origin = useMemo(() => getTxOrigin(app), [app])
 
+  // TODO: Need to use the SafeTxProvider here
   return (
-    <SignOrExecuteForm safeTx={safeTx} onSubmit={handleSubmit} error={safeTxError || submitError} origin={origin}>
+    <SignOrExecuteForm onSubmit={handleSubmit} origin={origin}>
       <>
         <ErrorBoundary fallback={<div>Error parsing data</div>}>
           <ApprovalEditor txs={txList} updateTxs={setTxList} />

src/components/settings/ContractVersion/UpdateSafeDialog.tsx

@@ -55,8 +55,9 @@ const ReviewUpdateSafeStep = ({ onSubmit }: { onSubmit: () => void }) => {
     return createMultiSendCallOnlyTx(txs)
   }, [chain, safe, safeLoaded])
 
+  // TODO: Need to use the SafeTxProvider here
   return (
-    <SignOrExecuteForm safeTx={safeTx} onSubmit={onSubmit} error={safeTxError}>
+    <SignOrExecuteForm onSubmit={onSubmit}>
       <Typography mb={2}>
         Update now to take advantage of new features and the highest security standards available.
       </Typography>

src/components/settings/SafeModules/RemoveModule/steps/ReviewRemoveModule.tsx

@@ -26,8 +26,9 @@ export const ReviewRemoveModule = ({ data, onSubmit }: { data: RemoveModuleData;
     onSubmit()
   }
 
+  // TODO: Need to use the SafeTxProvider here
   return (
-    <SignOrExecuteForm safeTx={safeTx} onSubmit={onFormSubmit} error={safeTxError}>
+    <SignOrExecuteForm onSubmit={onFormSubmit}>
       <SendToBlock address={data.address} title="Module" />
       <Typography my={2}>
         After removing this module, any feature or app that uses this module might no longer work. If this Safe Account

src/components/settings/SpendingLimits/NewSpendingLimit/steps/ReviewSpendingLimit.tsx

@@ -61,8 +61,9 @@ export const ReviewSpendingLimit = ({ data, onSubmit }: Props) => {
     onSubmit()
   }
 
+  // TODO: Need to use the SafeTxProvider here
   return (
-    <SignOrExecuteForm safeTx={safeTx} onSubmit={onFormSubmit} error={safeTxError}>
+    <SignOrExecuteForm onSubmit={onFormSubmit}>
       {token && (
         <TokenTransferReview amount={data.amount} tokenInfo={token.tokenInfo}>
           {!!existingSpendingLimit && (

src/components/settings/SpendingLimits/RemoveSpendingLimit/index.tsx

@@ -41,8 +41,9 @@ export const RemoveSpendingLimit = ({ data, onSubmit }: { data: SpendingLimitSta
     onSubmit()
   }
 
+  // TODO: Need to use the SafeTxProvider here
   return (
-    <SignOrExecuteForm safeTx={safeTx} onSubmit={onFormSubmit} error={safeTxError}>
+    <SignOrExecuteForm onSubmit={onFormSubmit}>
       {token && (
         <TokenTransferReview
           amount={safeFormatUnits(data.amount, token.tokenInfo.decimals)}

src/components/settings/TransactionGuards/RemoveGuard/steps/ReviewRemoveGuard.tsx

@@ -27,8 +27,9 @@ export const ReviewRemoveGuard = ({ data, onSubmit }: { data: RemoveGuardData; o
     onSubmit()
   }
 
+  // TODO: Need to use the SafeTxProvider here
   return (
-    <SignOrExecuteForm safeTx={safeTx} onSubmit={onFormSubmit} error={safeTxError}>
+    <SignOrExecuteForm onSubmit={onFormSubmit}>
       <Typography sx={({ palette }) => ({ color: palette.primary.light })}>Transaction guard</Typography>
       <EthHashInfo address={data.address} showCopyButton hasExplorer shortAddress={false} />
       <Typography my={2}>

src/components/settings/owner/AddOwnerDialog/DialogSteps/ReviewOwnerTxStep.tsx

@@ -54,8 +54,9 @@ export const ReviewOwnerTxStep = ({ data, onSubmit }: { data: ChangeOwnerData; o
     onSubmit()
   }
 
+  // TODO: Need to use the SafeTxProvider here
   return (
-    <SignOrExecuteForm safeTx={safeTx} onSubmit={addAddressBookEntryAndSubmit} error={safeTxError}>
+    <SignOrExecuteForm onSubmit={addAddressBookEntryAndSubmit}>
       <Grid
         container
         mt={-3}

src/components/settings/owner/ChangeThresholdDialog/index.tsx

@@ -114,12 +114,8 @@ const ChangeThresholdStep = ({ data, onSubmit }: { data: ChangeThresholdData; on
       </DialogContent>
 
       <Box mt={-5}>
-        <SignOrExecuteForm
-          safeTx={safeTx}
-          onSubmit={onChangeThreshold}
-          error={safeTxError}
-          disableSubmit={isSameThreshold}
-        />
+        {/* TODO: Need to use the SafeTxProvider here */}
+        <SignOrExecuteForm onSubmit={onChangeThreshold} disableSubmit={isSameThreshold} />
       </Box>
     </>
   )

src/components/settings/owner/RemoveOwnerDialog/DialogSteps/ReviewRemoveOwnerTxStep.tsx

@@ -30,8 +30,9 @@ export const ReviewRemoveOwnerTxStep = ({ data, onSubmit }: { data: RemoveOwnerD
     onSubmit()
   }
 
+  // TODO: Need to use the SafeTxProvider here
   return (
-    <SignOrExecuteForm safeTx={safeTx} onSubmit={onFormSubmit} error={safeTxError}>
+    <SignOrExecuteForm onSubmit={onFormSubmit}>
       <Grid
         container
         mt={-3}

src/components/tx/modals/NftBatchModal/ReviewNftBatch.tsx

@@ -25,8 +25,9 @@ const ReviewNftBatch = ({ params, onSubmit }: ReviewNftBatchProps): ReactElement
     return calls.length > 1 ? createMultiSendCallOnlyTx(calls) : createTx(calls[0])
   }, [safeAddress, params])
 
+  // TODO: Need to use the SafeTxProvider here
   return (
-    <SignOrExecuteForm safeTx={safeTx} onSubmit={onSubmit} error={safeTxError}>
+    <SignOrExecuteForm onSubmit={onSubmit}>
       <SendFromBlock title={`Sending ${tokens.length} NFT${tokens.length > 1 ? 's' : ''} from`} />
 
       <SendToBlock address={params.recipient} title="To" />

src/components/tx/modals/RejectTxModal/RejectTx.tsx

@@ -16,8 +16,9 @@ const RejectTx = ({ txNonce, onSubmit }: RejectTxProps): ReactElement => {
     return createRejectTx(txNonce)
   }, [txNonce])
 
+  // TODO: Need to use the SafeTxProvider here
   return (
-    <SignOrExecuteForm safeTx={rejectTx} isRejection onSubmit={onSubmit} error={rejectError}>
+    <SignOrExecuteForm isRejection onSubmit={onSubmit}>
       <Typography mb={2}>
         To reject the transaction, a separate rejection transaction will be created to replace the original one.
       </Typography>

src/components/tx/modals/TokenTransferModal/ReviewMultisigTx.tsx

@@ -24,8 +24,9 @@ const ReviewMultisigTx = ({ params, onSubmit }: TokenTransferModalProps): ReactE
     return createTx(txParams, params.txNonce)
   }, [params, decimals, address])
 
+  // TODO: Need to use the SafeTxProvider here
   return (
-    <SignOrExecuteForm safeTx={safeTx} onSubmit={onSubmit} error={safeTxError}>
+    <SignOrExecuteForm onSubmit={onSubmit}>
       {token && <TokenTransferReview amount={params.amount} tokenInfo={token.tokenInfo} />}
 
       <SendFromBlock />

src/components/tx/modals/TokenTransferModal/ReviewSpendingLimitTx.tsx

@@ -109,12 +109,7 @@ const ReviewSpendingLimitTx = ({ params, onSubmit }: TokenTransferModalProps): R
 
         <SendToBlock address={params.recipient} />
 
-        <AdvancedParams
-          params={advancedParams}
-          willExecute={true}
-          nonceReadonly={false}
-          onFormSubmit={setManualParams}
-        />
+        <AdvancedParams params={advancedParams} willExecute={true} onFormSubmit={setManualParams} />
 
         <WrongChainWarning />