fix: replace defaultProps with default parameters in SpreadsheetImport (#7406)

This PR addresses https://github.com/twentyhq/twenty/issues/6827

React has deprecated the use of `defaultProps` on function components
and will remove support in a future major release. This commit replaces
the usage of `defaultProps` in the `SpreadsheetImport` component with
default parameters to fix the following warning:

**Changes:**

- Removed `SpreadsheetImport.defaultProps =
defaultSpreadsheetImportProps;`
- Merged `defaultSpreadsheetImportProps` with incoming `props` using
object spread syntax.
- Adjusted the component to use the merged props (`mergedProps`) instead
of `props`.
This commit is contained in:
Gaz Jones 2024-10-02 19:30:39 +01:00 committed by GitHub
parent 83e43366bb
commit b39d262058
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -24,13 +24,16 @@ export const defaultSpreadsheetImportProps: Partial<
export const SpreadsheetImport = <T extends string>(
props: SpreadsheetImportProps<T>,
) => {
const mergedProps = {
...defaultSpreadsheetImportProps,
...props,
} as SpreadsheetImportProps<T>;
return (
<ReactSpreadsheetImportContextProvider values={props}>
<ModalWrapper isOpen={props.isOpen} onClose={props.onClose}>
<ReactSpreadsheetImportContextProvider values={mergedProps}>
<ModalWrapper isOpen={mergedProps.isOpen} onClose={mergedProps.onClose}>
<SpreadsheetImportStepperContainer />
</ModalWrapper>
</ReactSpreadsheetImportContextProvider>
);
};
SpreadsheetImport.defaultProps = defaultSpreadsheetImportProps;