sirtimbly/IronCalc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

FIX: onDelete and onCancel are mandatory properties

Browse Source 
In general optional arguments are a bad idea, because you loose the type
and all it's benefits.

Did you forget an argument or you didn't need it?
This commit is contained in:
Nicolás Hatcher
2025-01-01 12:18:29 +01:00
committed by Nicolás Hatcher Andrés
parent ddc785e7a6
commit 99d42cb1e2
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
webapp/src/components/NameManagerDialog/NamedRangeActive.tsx
View File

@@ -18,8 +18,8 @@ interface NamedRangeProperties {
scope?: number; scope?: number;
formula: string; formula: string;
onSave: () => void; onSave: () => void;
onDelete?: () => void; onDelete: () => void;
onCancel?: () => void; onCancel: () => void;
} }
function NamedRangeActive(properties: NamedRangeProperties) { function NamedRangeActive(properties: NamedRangeProperties) {