From 2bbc8ee117578e84e0bdbf4d0521435261620366 Mon Sep 17 00:00:00 2001 From: mertmit Date: Tue, 23 Aug 2022 21:33:12 +0300 Subject: [PATCH] fix: use promise instead of callback for validation Signed-off-by: mertmit --- packages/nc-gui-v2/utils/validation.ts | 20 +++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/packages/nc-gui-v2/utils/validation.ts b/packages/nc-gui-v2/utils/validation.ts index 1972873b99..45255a20fa 100644 --- a/packages/nc-gui-v2/utils/validation.ts +++ b/packages/nc-gui-v2/utils/validation.ts @@ -129,15 +129,17 @@ export const importExcelUrlValidator = { } export const extraParameterValidator = { - validator: (rule: any, value: any, callback: (errMsg?: string) => void) => { - for (const param of value) { - if (param.key === '') { - callback('Parameter key cannot be empty') - } - if (value.filter((el: any) => el.key === param.key).length !== 1) { - callback('Duplicate parameter keys are not allowed') + validator: (rule: any, value: any) => { + return new Promise((resolve, reject) => { + for (const param of value) { + if (param.key === '') { + return reject(new Error('Parameter key cannot be empty')) + } + if (value.filter((el: any) => el.key === param.key).length !== 1) { + return reject(new Error('Duplicate parameter keys are not allowed')) + } } - } - callback() + return resolve(true) + }) }, }