diff --git a/.eslintrc.js b/.eslintrc.js
index bf7bb15cb140a6118a2d2414942e0f495045b591..53796782da5eefcb643b835a28335dac42eb4119 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -57,7 +57,6 @@ module.exports = {
'@typescript-eslint/no-unused-vars': [
'warn',
{
- args: 'none',
varsIgnorePattern: '^_',
ignoreRestSiblings: true,
},
diff --git a/packages/desktop-client/src/browser-preload.browser.js b/packages/desktop-client/src/browser-preload.browser.js
index 37bd1f537192dc401995f6193191d436c4b9f482..ca5bc118d903b7f89e6a0a891a0b98aae4e35c7d 100644
--- a/packages/desktop-client/src/browser-preload.browser.js
+++ b/packages/desktop-client/src/browser-preload.browser.js
@@ -51,7 +51,7 @@ global.Actual = {
window.location.reload();
},
- openFileDialog: async ({ filters = [], properties }) => {
+ openFileDialog: async ({ filters = [] }) => {
return new Promise(resolve => {
let createdElement = false;
// Attempt to reuse an already-created file input.
@@ -91,7 +91,7 @@ global.Actual = {
.uploadFile(filename, ev.target.result)
.then(() => resolve([filepath]));
};
- reader.onerror = function (ev) {
+ reader.onerror = function () {
alert('Error reading file');
};
}
@@ -107,7 +107,7 @@ global.Actual = {
});
},
- saveFile: (contents, defaultFilename, dialogTitle) => {
+ saveFile: (contents, defaultFilename) => {
const temp = document.createElement('a');
temp.style = 'display: none';
temp.download = defaultFilename;
@@ -121,9 +121,9 @@ global.Actual = {
openURLInBrowser: url => {
window.open(url, '_blank');
},
- onEventFromMain: (type, handler) => {},
+ onEventFromMain: () => {},
applyAppUpdate: () => {},
- updateAppMenu: isBudgetOpen => {},
+ updateAppMenu: () => {},
ipcConnect: () => {},
getServerSocket: async () => {
diff --git a/packages/desktop-client/src/components/FatalError.tsx b/packages/desktop-client/src/components/FatalError.tsx
index f15b6852610225eeec38c546ccd43f4844eccf58..ebed6c72ee6946cefa8090d41fe621ec0abddb31 100644
--- a/packages/desktop-client/src/components/FatalError.tsx
+++ b/packages/desktop-client/src/components/FatalError.tsx
@@ -131,7 +131,7 @@ function SharedArrayBufferOverride() {
>
<Checkbox
checked={understand}
- onChange={_ => setUnderstand(!understand)}
+ onChange={() => setUnderstand(!understand)}
/>{' '}
I understand the risks, run Actual in the unsupported fallback mode
</label>
diff --git a/packages/desktop-client/src/components/FixedSizeList.tsx b/packages/desktop-client/src/components/FixedSizeList.tsx
index 2e74b00d5f80b9570b8afab5a41358e0cae84eb9..b2257a0a099dcb8f645dd0cf1a1964d53e82029c 100644
--- a/packages/desktop-client/src/components/FixedSizeList.tsx
+++ b/packages/desktop-client/src/components/FixedSizeList.tsx
@@ -2,7 +2,6 @@
import {
createRef,
PureComponent,
- type ReactElement,
type ReactNode,
type Ref,
type MutableRefObject,
@@ -11,7 +10,6 @@ import {
import memoizeOne from 'memoize-one';
-import { useResizeObserver } from '../hooks/useResizeObserver';
import { type CSSProperties } from '../style';
import { View } from './common/View';
@@ -20,16 +18,6 @@ const IS_SCROLLING_DEBOUNCE_INTERVAL = 150;
const defaultItemKey: FixedSizeListProps['itemKey'] = (index: number) => index;
-type ResizeObserverProps = {
- onResize: Parameters<typeof useResizeObserver>[0];
- children: (ref: Ref<HTMLDivElement>) => ReactElement;
-};
-
-function ResizeObserver({ onResize, children }: ResizeObserverProps) {
- const ref = useResizeObserver(onResize);
- return children(ref);
-}
-
type FixedSizeListProps = {
className?: string;
direction?: 'rtl' | 'ltr';
@@ -262,33 +250,29 @@ export class FixedSizeList extends PureComponent<
const estimatedTotalSize = this.getEstimatedTotalSize();
return (
- <ResizeObserver onResize={this.onHeaderResize}>
- {headerRef => (
- <div
- className={className}
- onScroll={this._onScrollVertical}
- ref={this._outerRefSetter}
- style={{
- height,
- width,
- overflow: 'hidden auto',
- }}
- >
- <View innerRef={headerRef}>{header}</View>
- <div
- ref={innerRef}
- style={{
- position: 'relative',
- height: estimatedTotalSize,
- width: '100%',
- pointerEvents: isScrolling ? 'none' : undefined,
- }}
- >
- {items}
- </div>
- </div>
- )}
- </ResizeObserver>
+ <div
+ className={className}
+ onScroll={this._onScrollVertical}
+ ref={this._outerRefSetter}
+ style={{
+ height,
+ width,
+ overflow: 'hidden auto',
+ }}
+ >
+ <View>{header}</View>
+ <div
+ ref={innerRef}
+ style={{
+ position: 'relative',
+ height: estimatedTotalSize,
+ width: '100%',
+ pointerEvents: isScrolling ? 'none' : undefined,
+ }}
+ >
+ {items}
+ </div>
+ </div>
);
}
@@ -305,10 +289,6 @@ export class FixedSizeList extends PureComponent<
}
};
- onHeaderResize = (rect: { height: number }) => {
- // this.setState({ headerHeight: rect.height });
- };
-
anchor() {
const itemKey = this.props.itemKey || defaultItemKey;
@@ -499,6 +479,7 @@ export class FixedSizeList extends PureComponent<
return style;
};
+ // eslint-disable-next-line @typescript-eslint/no-unused-vars
_getItemStyleCache = memoizeOne((_, __, ___) => ({}));
_getRangeToRender() {
diff --git a/packages/desktop-client/src/components/ManageRules.tsx b/packages/desktop-client/src/components/ManageRules.tsx
index 6e82621f92f5c523b25b99e04f7545694590d6da..0bcbfe254b42e405b535a273dbb7ff34558d7df4 100644
--- a/packages/desktop-client/src/components/ManageRules.tsx
+++ b/packages/desktop-client/src/components/ManageRules.tsx
@@ -230,7 +230,7 @@ function ManageRulesContent({
dispatch(
pushModal('edit-rule', {
rule,
- onSave: async newRule => {
+ onSave: async () => {
await loadRules();
setLoading(false);
},
diff --git a/packages/desktop-client/src/components/accounts/Account.jsx b/packages/desktop-client/src/components/accounts/Account.jsx
index 37653a025b81d6de294311e56c2be47945431422..85c5867206bcf94e4739378014947885e546205b 100644
--- a/packages/desktop-client/src/components/accounts/Account.jsx
+++ b/packages/desktop-client/src/components/accounts/Account.jsx
@@ -227,7 +227,7 @@ class AccountInternal extends PureComponent {
}
};
- const onUndo = async ({ tables, messages, undoTag }) => {
+ const onUndo = async ({ tables, messages }) => {
await maybeRefetch(tables);
// If all the messages are dealing with transactions, find the
@@ -516,7 +516,7 @@ class AccountInternal extends PureComponent {
});
}
},
- mappedData => {
+ () => {
return data;
},
);
diff --git a/packages/desktop-client/src/components/accounts/MobileAccount.jsx b/packages/desktop-client/src/components/accounts/MobileAccount.jsx
index aaaed0db57e5aeddbdc3a8cf142403d3f66be5b9..b8d4bb6ce772263468d6e385c6217c6b889e1288 100644
--- a/packages/desktop-client/src/components/accounts/MobileAccount.jsx
+++ b/packages/desktop-client/src/components/accounts/MobileAccount.jsx
@@ -43,7 +43,7 @@ const getSchedulesTransform = memoizeOne((id, hasSearch) => {
};
});
-function PreviewTransactions({ accountId, children }) {
+function PreviewTransactions({ children }) {
const scheduleData = useCachedSchedules();
if (scheduleData == null) {
diff --git a/packages/desktop-client/src/components/budget/MobileBudget.tsx b/packages/desktop-client/src/components/budget/MobileBudget.tsx
index 2a426909a4c0a5305687d69523e3f9656881ea31..b1b59f76c8cd0e7e0b2c9df464ca3c2d4d46e89a 100644
--- a/packages/desktop-client/src/components/budget/MobileBudget.tsx
+++ b/packages/desktop-client/src/components/budget/MobileBudget.tsx
@@ -408,19 +408,15 @@ function BudgetInner(props: BudgetInnerProps) {
await sync();
}}
>
- {({ refreshing, onRefresh }) => (
+ {({ onRefresh }) => (
<BudgetTable
// This key forces the whole table rerender when the number
// format changes
key={numberFormat + hideFraction}
categoryGroups={categoryGroups}
- categories={categories}
type={budgetType}
month={currentMonth}
monthBounds={bounds}
- // refreshControl={
- // <RefreshControl refreshing={refreshing} onRefresh={onRefresh} />
- // }
editMode={editMode}
onEditMode={flag => setEditMode(flag)}
onShowBudgetSummary={onShowBudgetSummary}
@@ -438,9 +434,6 @@ function BudgetInner(props: BudgetInnerProps) {
onBudgetAction={applyBudgetAction}
onRefresh={onRefresh}
onSwitchBudgetType={onSwitchBudgetType}
- onSaveNotes={onSaveNotes}
- onEditGroupNotes={onEditGroupNotes}
- onEditCategoryNotes={onEditCategoryNotes}
savePrefs={savePrefs}
pushModal={pushModal}
onEditGroup={onEditGroup}
diff --git a/packages/desktop-client/src/components/budget/MobileBudgetTable.jsx b/packages/desktop-client/src/components/budget/MobileBudgetTable.jsx
index 3c2251d77b3679b47f48c6e9f030c34b0b8f9b8e..cb75b90eef9cc7efaa5aad3e48663737b6f8c159 100644
--- a/packages/desktop-client/src/components/budget/MobileBudgetTable.jsx
+++ b/packages/desktop-client/src/components/budget/MobileBudgetTable.jsx
@@ -145,7 +145,7 @@ function BudgetCell({
});
}
- function onAmountClick(e) {
+ function onAmountClick() {
onEdit?.(categoryId);
}
@@ -243,7 +243,6 @@ const ExpenseCategory = memo(function ExpenseCategory({
blank,
style,
month,
- editMode,
onEdit,
isEditingBudget,
onEditBudget,
@@ -569,7 +568,6 @@ const IncomeGroupTotals = memo(function IncomeGroupTotals({
budgeted,
balance,
style,
- editMode,
onEdit,
}) {
const listItemRef = useRef();
@@ -657,7 +655,6 @@ const IncomeCategory = memo(function IncomeCategory({
balance,
month,
style,
- editMode,
onEdit,
onBudgetAction,
isEditingBudget,
@@ -1122,7 +1119,6 @@ function BudgetGroups({
export function BudgetTable({
type,
categoryGroups,
- categories,
month,
monthBounds,
editMode,
@@ -1143,9 +1139,6 @@ export function BudgetTable({
onBudgetAction,
onRefresh,
onSwitchBudgetType,
- onSaveNotes,
- onEditGroupNotes,
- onEditCategoryNotes,
savePrefs,
pushModal,
onEditGroup,
@@ -1154,7 +1147,6 @@ export function BudgetTable({
onEditCategoryBudget,
openBalanceActionMenuId,
onOpenBalanceActionMenu,
- ...props
}) {
const { width } = useResponsive();
const show3Cols = width >= 360;
diff --git a/packages/desktop-client/src/components/budget/SidebarCategory.tsx b/packages/desktop-client/src/components/budget/SidebarCategory.tsx
index 04bd471d24b9ddede361dff84873a20868bc8089..8ca35fde6b9f860dacc3ccb7bb94978d13edc3a6 100644
--- a/packages/desktop-client/src/components/budget/SidebarCategory.tsx
+++ b/packages/desktop-client/src/components/budget/SidebarCategory.tsx
@@ -154,7 +154,7 @@ export function SidebarCategory({
>
<InputCell
value={category.name}
- formatter={value => displayed}
+ formatter={() => displayed}
width="flex"
exposed={editing || temporary}
onUpdate={value => {
diff --git a/packages/desktop-client/src/components/budget/SidebarGroup.tsx b/packages/desktop-client/src/components/budget/SidebarGroup.tsx
index c298e3d28e37690dae22154a140d63829ea04683..bbdd36b360304b75c90021b4532dcaf8988d0eb8 100644
--- a/packages/desktop-client/src/components/budget/SidebarGroup.tsx
+++ b/packages/desktop-client/src/components/budget/SidebarGroup.tsx
@@ -27,7 +27,6 @@ type SidebarGroupProps = {
collapsed: boolean;
dragPreview?: boolean;
innerRef?: ConnectDragSource;
- borderColor?: string;
style?: CSSProperties;
onEdit?: (id: string) => void;
onSave?: (group: object) => Promise<void>;
@@ -44,7 +43,6 @@ export function SidebarGroup({
dragPreview,
innerRef,
style,
- borderColor = theme.tableBorder,
onEdit,
onSave,
onDelete,
@@ -63,7 +61,7 @@ export function SidebarGroup({
userSelect: 'none',
WebkitUserSelect: 'none',
}}
- onClick={e => {
+ onClick={() => {
onToggleCollapse(group.id);
}}
>
@@ -181,7 +179,7 @@ export function SidebarGroup({
>
<InputCell
value={group.name}
- formatter={value => displayed}
+ formatter={() => displayed}
width="flex"
exposed={editing}
onUpdate={value => {
diff --git a/packages/desktop-client/src/components/budget/index.tsx b/packages/desktop-client/src/components/budget/index.tsx
index 884567a27b58eb1fb0b9f6857075212502426e8b..3363140e6f73148adb2d0f645537461946cfd6f5 100644
--- a/packages/desktop-client/src/components/budget/index.tsx
+++ b/packages/desktop-client/src/components/budget/index.tsx
@@ -533,7 +533,7 @@ const RolloverBudgetSummary = memo<{ month: string }>(props => {
);
});
-export function Budget(props) {
+export function Budget() {
const startMonth = useSelector(
state => state.prefs.local['budget.startMonth'],
);
diff --git a/packages/desktop-client/src/components/budget/report/BalanceTooltip.tsx b/packages/desktop-client/src/components/budget/report/BalanceTooltip.tsx
index 76eddb31928b2e9e86225a18a4351f7caa2a9e7e..bf806d12b9067813f55c8cbf5b5309c41b21bc5f 100644
--- a/packages/desktop-client/src/components/budget/report/BalanceTooltip.tsx
+++ b/packages/desktop-client/src/components/budget/report/BalanceTooltip.tsx
@@ -38,7 +38,7 @@ export function BalanceTooltip({
{...tooltipProps}
>
<Menu
- onMenuSelect={type => {
+ onMenuSelect={() => {
onBudgetAction(monthIndex, 'carryover', {
category: categoryId,
flag: !carryover,
diff --git a/packages/desktop-client/src/components/budget/rollover/CoverTooltip.tsx b/packages/desktop-client/src/components/budget/rollover/CoverTooltip.tsx
index bd822c9cb964ea2d89aa478f6d8fba70763005b9..2f33e371f7d918c5671885264083dbf1f4d26291 100644
--- a/packages/desktop-client/src/components/budget/rollover/CoverTooltip.tsx
+++ b/packages/desktop-client/src/components/budget/rollover/CoverTooltip.tsx
@@ -48,7 +48,7 @@ export function CoverTooltip({
categoryGroups={categoryGroups}
value={null}
openOnFocus={true}
- onUpdate={id => {}}
+ onUpdate={() => {}}
onSelect={id => setCategory(id)}
inputProps={{
inputRef: node,
diff --git a/packages/desktop-client/src/components/budget/rollover/TransferTooltip.tsx b/packages/desktop-client/src/components/budget/rollover/TransferTooltip.tsx
index 8671a1261c79f6e0ecb89884688cc120e409c57a..0d27946f1387b3a3427561c66e64fffc3b82ad73 100644
--- a/packages/desktop-client/src/components/budget/rollover/TransferTooltip.tsx
+++ b/packages/desktop-client/src/components/budget/rollover/TransferTooltip.tsx
@@ -98,7 +98,7 @@ export function TransferTooltip({
categoryGroups={categoryGroups}
value={null}
openOnFocus={true}
- onUpdate={id => {}}
+ onUpdate={() => {}}
onSelect={id => setCategory(id)}
inputProps={{ onEnter: submit, placeholder: '(none)' }}
/>
diff --git a/packages/desktop-client/src/components/budget/rollover/budgetsummary/BudgetSummary.tsx b/packages/desktop-client/src/components/budget/rollover/budgetsummary/BudgetSummary.tsx
index dc66bed7a9f938b2ecb25adcdd26ed4fe1382165..209ab5e31be7237bcfbfcd5d12843c05ccd7fe2b 100644
--- a/packages/desktop-client/src/components/budget/rollover/budgetsummary/BudgetSummary.tsx
+++ b/packages/desktop-client/src/components/budget/rollover/budgetsummary/BudgetSummary.tsx
@@ -35,7 +35,7 @@ export function BudgetSummary({
} = useRollover();
const [menuOpen, setMenuOpen] = useState(false);
- function onMenuOpen(e) {
+ function onMenuOpen() {
setMenuOpen(true);
}
diff --git a/packages/desktop-client/src/components/common/Menu.tsx b/packages/desktop-client/src/components/common/Menu.tsx
index dc67988c626c61f552d4d1076f743149cf4090ad..27c428aa339bbd2a1d83210cf010d03a41b36bf1 100644
--- a/packages/desktop-client/src/components/common/Menu.tsx
+++ b/packages/desktop-client/src/components/common/Menu.tsx
@@ -163,7 +163,7 @@ export function Menu<T extends MenuItem>({
}}
onMouseEnter={() => setHoveredIndex(idx)}
onMouseLeave={() => setHoveredIndex(null)}
- onClick={e =>
+ onClick={() =>
!item.disabled && onMenuSelect && onMenuSelect(item.name)
}
>
diff --git a/packages/desktop-client/src/components/modals/CategoryGroupMenu.tsx b/packages/desktop-client/src/components/modals/CategoryGroupMenu.tsx
index bd8d3e5d11278db4326865762a52bc5ac4d6d9ee..6a1368b5a4b8a0a3e16c34f941e99f10abf195c4 100644
--- a/packages/desktop-client/src/components/modals/CategoryGroupMenu.tsx
+++ b/packages/desktop-client/src/components/modals/CategoryGroupMenu.tsx
@@ -137,7 +137,7 @@ export function CategoryGroupMenu({
notes={notes?.length > 0 ? notes : 'No notes'}
editable={false}
focused={false}
- getStyle={editable => ({
+ getStyle={() => ({
...styles.mediumText,
borderRadius: 6,
...((!notes || notes.length === 0) && {
diff --git a/packages/desktop-client/src/components/modals/CategoryMenu.tsx b/packages/desktop-client/src/components/modals/CategoryMenu.tsx
index f42bb96904bbc3ab9abd0cd9b4bddf1729280485..2e2b79181f3fd01ba895c1b1b64515b6fdea3099 100644
--- a/packages/desktop-client/src/components/modals/CategoryMenu.tsx
+++ b/packages/desktop-client/src/components/modals/CategoryMenu.tsx
@@ -128,7 +128,7 @@ export function CategoryMenu({
notes={originalNotes?.length > 0 ? originalNotes : 'No notes'}
editable={false}
focused={false}
- getStyle={editable => ({
+ getStyle={() => ({
borderRadius: 6,
...((!originalNotes || originalNotes.length === 0) && {
justifySelf: 'center',
diff --git a/packages/desktop-client/src/components/modals/EditRule.jsx b/packages/desktop-client/src/components/modals/EditRule.jsx
index 661cc799e8c753b824e9b9ee46e60eb474efe351..4d42814e53dd5dd07206d076f144883706a475c2 100644
--- a/packages/desktop-client/src/components/modals/EditRule.jsx
+++ b/packages/desktop-client/src/components/modals/EditRule.jsx
@@ -122,7 +122,7 @@ export function OpSelect({
);
}
-function EditorButtons({ onAdd, onDelete, style }) {
+function EditorButtons({ onAdd, onDelete }) {
return (
<>
{onDelete && (
@@ -310,7 +310,7 @@ const actionFields = [
'date',
'amount',
].map(field => [field, mapField(field)]);
-function ActionEditor({ ops, action, editorStyle, onChange, onDelete, onAdd }) {
+function ActionEditor({ action, editorStyle, onChange, onDelete, onAdd }) {
const { field, op, value, type, error, inputKey = 'initial' } = action;
return (
diff --git a/packages/desktop-client/src/components/modals/Notes.tsx b/packages/desktop-client/src/components/modals/Notes.tsx
index e36f2315217c97a75793ab7fe1111f12b0d6d41c..76516325466031d8522d5cbea4d4344d9ae02cf8 100644
--- a/packages/desktop-client/src/components/modals/Notes.tsx
+++ b/packages/desktop-client/src/components/modals/Notes.tsx
@@ -63,7 +63,7 @@ export function Notes({ modalProps, id, name, onSave }: NotesProps) {
notes={notes}
editable={true}
focused={true}
- getStyle={editable => ({
+ getStyle={() => ({
borderRadius: 6,
flex: 1,
minWidth: 0,
diff --git a/packages/desktop-client/src/components/modals/SingleInput.tsx b/packages/desktop-client/src/components/modals/SingleInput.tsx
index 8c4daef9451f39cf325b8f905329f6f17cc9949c..89f4b33479920b18254bb9fe4a0183b247fa45ca 100644
--- a/packages/desktop-client/src/components/modals/SingleInput.tsx
+++ b/packages/desktop-client/src/components/modals/SingleInput.tsx
@@ -81,7 +81,7 @@ export function SingleInput({
...styles.mediumText,
flexBasis: '50%',
}}
- onPointerUp={e => _onSubmit(value)}
+ onPointerUp={() => _onSubmit(value)}
>
{buttonText}
</Button>
diff --git a/packages/desktop-client/src/components/payees/ManagePayeesWithData.jsx b/packages/desktop-client/src/components/payees/ManagePayeesWithData.jsx
index 338adbb08cdddb5283b4aae65dc32595c400aaf2..a828c5e73f2774647cf4cc3253b63a875409ee8f 100644
--- a/packages/desktop-client/src/components/payees/ManagePayeesWithData.jsx
+++ b/packages/desktop-client/src/components/payees/ManagePayeesWithData.jsx
@@ -63,7 +63,7 @@ export function ManagePayeesWithData({ initialSelectedIds }) {
};
}, []);
- async function onUndo({ tables, messages, meta, url }, scroll = false) {
+ async function onUndo({ tables, messages, meta }) {
if (!tables.includes('payees') && !tables.includes('payee_mapping')) {
return;
}
@@ -83,7 +83,7 @@ export function ManagePayeesWithData({ initialSelectedIds }) {
useEffect(() => {
if (lastUndoState.current) {
- onUndo(lastUndoState.current, true);
+ onUndo(lastUndoState.current);
}
return listen('undo-event', onUndo);
diff --git a/packages/desktop-client/src/components/reports/ChooseGraph.tsx b/packages/desktop-client/src/components/reports/ChooseGraph.tsx
index a4e0b5b21bf6ad4a21493ef3f5f6ecd25ff56470..a3149c39119f30c1fd2dd9d509d1e29d3acf4888 100644
--- a/packages/desktop-client/src/components/reports/ChooseGraph.tsx
+++ b/packages/desktop-client/src/components/reports/ChooseGraph.tsx
@@ -26,7 +26,6 @@ type ChooseGraphProps = {
graphType: string;
balanceType: string;
groupBy: string;
- scrollWidth?: number;
setScrollWidth?: (value: number) => void;
months?: Month[];
viewLabels?: boolean;
@@ -40,7 +39,6 @@ export function ChooseGraph({
graphType,
balanceType,
groupBy,
- scrollWidth,
setScrollWidth,
months,
viewLabels,
@@ -139,7 +137,6 @@ export function ChooseGraph({
headerScrollRef={headerScrollRef}
handleScroll={handleScroll}
interval={mode === 'time' && data.monthData}
- scrollWidth={scrollWidth}
groupBy={groupBy}
balanceType={balanceType}
compact={compact}
@@ -158,7 +155,6 @@ export function ChooseGraph({
<ReportTableTotals
totalScrollRef={totalScrollRef}
handleScroll={handleScroll}
- scrollWidth={scrollWidth}
data={data}
mode={mode}
balanceTypeOp={balanceTypeOp}
diff --git a/packages/desktop-client/src/components/reports/Header.jsx b/packages/desktop-client/src/components/reports/Header.jsx
index 34188780b6a8c336f1cc2e7e807cadc1dcc2baf1..e3231f8d696eac8f7b29bd4ef32eefad3f2a0ecd 100644
--- a/packages/desktop-client/src/components/reports/Header.jsx
+++ b/packages/desktop-client/src/components/reports/Header.jsx
@@ -75,7 +75,6 @@ export function Header({
onDeleteFilter,
onCondOpChange,
headerPrefixItems,
- selectGraph,
}) {
const location = useLocation();
const path = location.pathname;
diff --git a/packages/desktop-client/src/components/reports/graphs/AreaGraph.tsx b/packages/desktop-client/src/components/reports/graphs/AreaGraph.tsx
index 26faf6cd888d1945bfa8c80c05700b4867517cc9..28b47e3527e212c607448d8adbac110ce6172e80 100644
--- a/packages/desktop-client/src/components/reports/graphs/AreaGraph.tsx
+++ b/packages/desktop-client/src/components/reports/graphs/AreaGraph.tsx
@@ -173,7 +173,7 @@ export function AreaGraph({
...(compact && { height: 'auto' }),
}}
>
- {(width, height, portalHost) =>
+ {(width, height) =>
data.monthData && (
<ResponsiveContainer>
<div>
diff --git a/packages/desktop-client/src/components/reports/graphs/BarGraph.tsx b/packages/desktop-client/src/components/reports/graphs/BarGraph.tsx
index e17171b56354f19b9bd81630c9e40e7358ea757f..7f4f9ec3bdd6f3335adc4ddb8dec213ce97e0f8c 100644
--- a/packages/desktop-client/src/components/reports/graphs/BarGraph.tsx
+++ b/packages/desktop-client/src/components/reports/graphs/BarGraph.tsx
@@ -167,7 +167,7 @@ export function BarGraph({
...(compact && { height: 'auto' }),
}}
>
- {(width, height, portalHost) =>
+ {(width, height) =>
data[splitData] && (
<ResponsiveContainer>
<div>
diff --git a/packages/desktop-client/src/components/reports/graphs/BarLineGraph.tsx b/packages/desktop-client/src/components/reports/graphs/BarLineGraph.tsx
index fe659df811b1947b18ace19c942edda810579a19..26e823288fb131c87943940459e349606092fc93 100644
--- a/packages/desktop-client/src/components/reports/graphs/BarLineGraph.tsx
+++ b/packages/desktop-client/src/components/reports/graphs/BarLineGraph.tsx
@@ -33,10 +33,9 @@ type PayloadItem = {
type CustomTooltipProps = {
active?: boolean;
payload?: PayloadItem[];
- label?: string;
};
-const CustomTooltip = ({ active, payload, label }: CustomTooltipProps) => {
+const CustomTooltip = ({ active, payload }: CustomTooltipProps) => {
if (active && payload && payload.length) {
return (
<div
@@ -88,7 +87,7 @@ export function BarLineGraph({ style, graphData, compact }: BarLineGraphProps) {
...(compact && { height: 'auto' }),
}}
>
- {(width, height, portalHost) =>
+ {(width, height) =>
graphData && (
<ResponsiveContainer>
<div>
diff --git a/packages/desktop-client/src/components/reports/graphs/DonutGraph.tsx b/packages/desktop-client/src/components/reports/graphs/DonutGraph.tsx
index 6a69bb2ec9bea1fa8a8c8c7e7e9f026d32ca9ede..6fb3ebc03f21103e68e6aef4bb5c5dfdc32c419e 100644
--- a/packages/desktop-client/src/components/reports/graphs/DonutGraph.tsx
+++ b/packages/desktop-client/src/components/reports/graphs/DonutGraph.tsx
@@ -157,7 +157,7 @@ export function DonutGraph({
...(compact && { height: 'auto' }),
}}
>
- {(width, height, portalHost) =>
+ {(width, height) =>
data[splitData] && (
<ResponsiveContainer>
<div>
diff --git a/packages/desktop-client/src/components/reports/graphs/LineGraph.tsx b/packages/desktop-client/src/components/reports/graphs/LineGraph.tsx
index 4cba1f3e74442130a730a056dc7df5e28264845a..819528439910b7dea30d22e2a40ae1e0dc7e01eb 100644
--- a/packages/desktop-client/src/components/reports/graphs/LineGraph.tsx
+++ b/packages/desktop-client/src/components/reports/graphs/LineGraph.tsx
@@ -32,10 +32,9 @@ type PayloadItem = {
type CustomTooltipProps = {
active?: boolean;
payload?: PayloadItem[];
- label?: string;
};
-const CustomTooltip = ({ active, payload, label }: CustomTooltipProps) => {
+const CustomTooltip = ({ active, payload }: CustomTooltipProps) => {
if (active && payload && payload.length) {
return (
<div
@@ -87,7 +86,7 @@ export function LineGraph({ style, graphData, compact }: LineGraphProps) {
...(compact && { height: 'auto' }),
}}
>
- {(width, height, portalHost) =>
+ {(width, height) =>
graphData && (
<ResponsiveContainer>
<div>
diff --git a/packages/desktop-client/src/components/reports/graphs/NetWorthGraph.tsx b/packages/desktop-client/src/components/reports/graphs/NetWorthGraph.tsx
index 9e9eb3fadd8b3f4085f225edbb3899bea30b214e..f3974f7e4eccc97a8650b2749097fb7bf9069377 100644
--- a/packages/desktop-client/src/components/reports/graphs/NetWorthGraph.tsx
+++ b/packages/desktop-client/src/components/reports/graphs/NetWorthGraph.tsx
@@ -22,16 +22,12 @@ type NetWorthGraphProps = {
style?: CSSProperties;
graphData;
compact: boolean;
- domain?: {
- y?: [number, number];
- };
};
export function NetWorthGraph({
style,
graphData,
compact,
- domain,
}: NetWorthGraphProps) {
const tickFormatter = tick => {
return `${Math.round(tick).toLocaleString()}`; // Formats the tick values as strings with commas
@@ -66,11 +62,10 @@ export function NetWorthGraph({
type CustomTooltipProps = {
active?: boolean;
payload?: PayloadItem[];
- label?: string;
};
// eslint-disable-next-line react/no-unstable-nested-components
- const CustomTooltip = ({ active, payload, label }: CustomTooltipProps) => {
+ const CustomTooltip = ({ active, payload }: CustomTooltipProps) => {
if (active && payload && payload.length) {
return (
<div
@@ -113,7 +108,7 @@ export function NetWorthGraph({
...(compact && { height: 'auto' }),
}}
>
- {(width, height, portalHost) =>
+ {(width, height) =>
graphData && (
<ResponsiveContainer>
<div>
diff --git a/packages/desktop-client/src/components/reports/graphs/SankeyGraph.tsx b/packages/desktop-client/src/components/reports/graphs/SankeyGraph.tsx
index d079a1a40e46fc714d3a95cc8734c9d7e7329b92..5f97bfcba804cad8ec8e4cf3e345c3009bca556e 100644
--- a/packages/desktop-client/src/components/reports/graphs/SankeyGraph.tsx
+++ b/packages/desktop-client/src/components/reports/graphs/SankeyGraph.tsx
@@ -114,7 +114,7 @@ export function SankeyGraph({ style, data, compact }: SankeyProps) {
...(compact && { height: 'auto' }),
}}
>
- {(width, height, portalHost) => (
+ {width => (
<ResponsiveContainer>
<Sankey
data={sankeyData}
diff --git a/packages/desktop-client/src/components/reports/graphs/StackedBarGraph.tsx b/packages/desktop-client/src/components/reports/graphs/StackedBarGraph.tsx
index 812be419ac03b5f6aed8bf3cbdff2feb4e430934..16b5846676994dcc4b4fcece67034ec8d3710161 100644
--- a/packages/desktop-client/src/components/reports/graphs/StackedBarGraph.tsx
+++ b/packages/desktop-client/src/components/reports/graphs/StackedBarGraph.tsx
@@ -142,7 +142,7 @@ export function StackedBarGraph({
...(compact && { height: 'auto' }),
}}
>
- {(width, height, portalHost) =>
+ {(width, height) =>
data.monthData && (
<ResponsiveContainer>
<div>
diff --git a/packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableHeader.tsx b/packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableHeader.tsx
index ea81dc5343814cad26668a827209e040ecb63186..4bb20248c3200c33f6d046e386a8e587a6ebc976 100644
--- a/packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableHeader.tsx
+++ b/packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableHeader.tsx
@@ -9,7 +9,6 @@ import { View } from '../../../common/View';
import { Row, Cell } from '../../../table';
type ReportTableHeaderProps = {
- scrollWidth?: number;
groupBy: string;
interval?: DataEntity[];
balanceType: string;
@@ -19,7 +18,6 @@ type ReportTableHeaderProps = {
};
export function ReportTableHeader({
- scrollWidth,
groupBy,
interval,
balanceType,
diff --git a/packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableTotals.tsx b/packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableTotals.tsx
index 784f6054e240e0c911bca71a3e9eeecb5f51d1c7..5153ae1b0baec0c0784713dc17cfb5474a445274 100644
--- a/packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableTotals.tsx
+++ b/packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableTotals.tsx
@@ -15,7 +15,6 @@ import { Row, Cell } from '../../../table';
type ReportTableTotalsProps = {
data: GroupedEntity;
- scrollWidth?: number;
balanceTypeOp: string;
mode: string;
monthsCount: number;
@@ -26,7 +25,6 @@ type ReportTableTotalsProps = {
export function ReportTableTotals({
data,
- scrollWidth,
balanceTypeOp,
mode,
monthsCount,
diff --git a/packages/desktop-client/src/components/reports/reports/CashFlow.tsx b/packages/desktop-client/src/components/reports/reports/CashFlow.tsx
index e1ad6356cb459a064d198dd5df0cedb0114ccce0..70f195efdab7925d95eec430ff2fcdaa9e5a74f7 100644
--- a/packages/desktop-client/src/components/reports/reports/CashFlow.tsx
+++ b/packages/desktop-client/src/components/reports/reports/CashFlow.tsx
@@ -110,7 +110,6 @@ export function CashFlow(): JSX.Element {
conditionsOp={conditionsOp}
onCondOpChange={onCondOpChange}
headerPrefixItems={undefined}
- selectGraph={undefined}
/>
<View
style={{
diff --git a/packages/desktop-client/src/components/reports/spreadsheets/cash-flow-spreadsheet.tsx b/packages/desktop-client/src/components/reports/spreadsheets/cash-flow-spreadsheet.tsx
index d2b804320558e4a0c3678ec7d210a7ed2e01736a..7048e0a1d6dfee5625a6be5a533cc6653c915921 100644
--- a/packages/desktop-client/src/components/reports/spreadsheets/cash-flow-spreadsheet.tsx
+++ b/packages/desktop-client/src/components/reports/spreadsheets/cash-flow-spreadsheet.tsx
@@ -58,7 +58,7 @@ export function cashFlowByDate(
});
const conditionsOpKey = conditionsOp === 'or' ? '$or' : '$and';
- function makeQuery(where) {
+ function makeQuery() {
const query = q('transactions')
.filter({
[conditionsOpKey]: filters,
@@ -99,8 +99,8 @@ export function cashFlowByDate(
'account.offbudget': false,
})
.calculate({ $sum: '$amount' }),
- makeQuery('amount > 0').filter({ amount: { $gt: 0 } }),
- makeQuery('amount < 0').filter({ amount: { $lt: 0 } }),
+ makeQuery().filter({ amount: { $gt: 0 } }),
+ makeQuery().filter({ amount: { $lt: 0 } }),
],
data => {
setData(recalculate(data, start, end, isConcise));
diff --git a/upcoming-release-notes/2256.md b/upcoming-release-notes/2256.md
new file mode 100644
index 0000000000000000000000000000000000000000..02d56a0c79fa8070a765883f081b0f42ffcd6003
--- /dev/null
+++ b/upcoming-release-notes/2256.md
@@ -0,0 +1,6 @@
+---
+category: Maintenance
+authors: [MatissJanis]
+---
+
+Code clean-up: removing unused variables (pt.2)