From 982871b61e887b6e81d9972d0a16cc914f238fdf Mon Sep 17 00:00:00 2001
From: Haider Alshamma <haidera@nulogy.com>
Date: Tue, 7 Jan 2025 15:47:50 -0500
Subject: [PATCH] fix: a more accurate type for the Select onChange prop

---
 src/Select/Select.tsx |  4 ++--
 src/Select/lib.ts     | 16 +++++++++-------
 2 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/src/Select/Select.tsx b/src/Select/Select.tsx
index 0b7ffea5d..44727fed1 100644
--- a/src/Select/Select.tsx
+++ b/src/Select/Select.tsx
@@ -22,7 +22,7 @@ import {
 } from "./SelectComponents";
 import { SelectOption } from "./SelectOption";
 import MenuList from "./MenuList";
-import { calcOptionsLength, checkOptionsAreValid, extractValue, getReactSelectValue } from "./lib";
+import { calcOptionsLength, checkOptionsAreValid, CustomOnChangeValue, extractValue, getReactSelectValue } from "./lib";
 
 export type NDSOptionValue = string | number | boolean | null;
 
@@ -46,7 +46,7 @@ interface CustomProps<Option extends NDSOption, IsMulti extends boolean, Group e
   defaultValue?: PropsValue<Option["value"]>;
   value?: PropsValue<Option["value"]>;
   options: readonly Option[];
-  onChange?: (newValue: PropsValue<Option["value"]>) => void;
+  onChange?: (newValue: CustomOnChangeValue<IsMulti>) => void;
   windowThreshold?: number;
   styles?: (selectStyles: StylesConfig<Option, IsMulti, Group>) => StylesConfig<Option, IsMulti, Group>;
 }
diff --git a/src/Select/lib.ts b/src/Select/lib.ts
index 844f82da4..2f7ba990b 100644
--- a/src/Select/lib.ts
+++ b/src/Select/lib.ts
@@ -1,5 +1,5 @@
 import * as React from "react";
-import { Options, PropsValue } from "react-select";
+import { OnChangeValue, Options, PropsValue } from "react-select";
 import { NDSOption, NDSOptionValue } from "./Select";
 
 export function calcOptionsLength(options) {
@@ -92,20 +92,22 @@ export function getReactSelectValue(options: Options<NDSOption>, input: PropsVal
   return getOption(options, input);
 }
 
-export function extractValue(
-  options: Options<NDSOption> | NDSOption,
-  isMulti: boolean
-): NDSOptionValue[] | NDSOptionValue {
+export type CustomOnChangeValue<IsMulti extends boolean> = IsMulti extends true ? NDSOptionValue[] : NDSOptionValue;
+
+export function extractValue<IsMulti extends boolean>(
+  options: OnChangeValue<NDSOption, IsMulti>,
+  isMulti: IsMulti
+): CustomOnChangeValue<IsMulti> {
   if (options === null) {
     return null;
   }
 
   if (!Array.isArray(options)) {
-    return (options as NDSOption).value;
+    return (options as NDSOption).value as CustomOnChangeValue<IsMulti>;
   }
 
   if (isMulti) {
-    return options && options.length ? options.map((o) => o.value) : [];
+    return (options && options.length ? options.map((o) => o.value) : []) as CustomOnChangeValue<IsMulti>;
   }
 
   throw new Error("UNEXPECTED ERROR: don't forget to enable isMulti");