Skip to content

Commit 966f81e

Browse files
linting fixes
1 parent b51fc2d commit 966f81e

File tree

2 files changed

+34
-20
lines changed

2 files changed

+34
-20
lines changed

packages/connect-react/src/hooks/form-context.tsx

Lines changed: 16 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -82,21 +82,27 @@ export const FormContextProvider = <T extends ConfigurableProps>({
8282
const {
8383
component, configuredProps: __configuredProps, propNames, externalUserId, userId, sdkResponse, enableDebugging,
8484
} = formProps;
85-
85+
8686
// Resolve user ID with deprecation warning
87-
const { resolvedId: resolvedExternalUserId, warningType } = useMemo(() =>
88-
resolveUserId(externalUserId, userId),
89-
[externalUserId, userId]
90-
);
87+
const {
88+
resolvedId: resolvedExternalUserId, warningType,
89+
} = useMemo(() =>
90+
resolveUserId(externalUserId, userId),
91+
[
92+
externalUserId,
93+
userId,
94+
]);
9195

9296
// Show deprecation warnings in useEffect to avoid render side effects
9397
useEffect(() => {
94-
if (warningType === 'both') {
95-
console.warn('[connect-react] Both externalUserId and userId provided. Using externalUserId. Please remove userId to avoid this warning.');
96-
} else if (warningType === 'deprecated') {
97-
console.warn('[connect-react] userId is deprecated. Please use externalUserId instead.');
98+
if (warningType === "both") {
99+
console.warn("[connect-react] Both externalUserId and userId provided. Using externalUserId. Please remove userId to avoid this warning.");
100+
} else if (warningType === "deprecated") {
101+
console.warn("[connect-react] userId is deprecated. Please use externalUserId instead.");
98102
}
99-
}, [warningType]);
103+
}, [
104+
warningType,
105+
]);
100106
const componentId = component.key;
101107

102108
const [

packages/connect-react/src/utils/resolve-user-id.ts

Lines changed: 18 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -3,19 +3,27 @@
33
* Prefers externalUserId and returns both the resolved value and warning info
44
*/
55
export const resolveUserId = (
6-
externalUserId?: string,
7-
userId?: string
8-
): { resolvedId: string; warningType?: 'both' | 'deprecated' } => {
6+
externalUserId?: string,
7+
userId?: string,
8+
): { resolvedId: string; warningType?: "both" | "deprecated" } => {
99
if (externalUserId) {
1010
if (userId) {
11-
return { resolvedId: externalUserId, warningType: 'both' };
11+
return {
12+
resolvedId: externalUserId,
13+
warningType: "both",
14+
};
1215
}
13-
return { resolvedId: externalUserId };
16+
return {
17+
resolvedId: externalUserId,
18+
};
1419
}
15-
20+
1621
if (userId) {
17-
return { resolvedId: userId, warningType: 'deprecated' };
22+
return {
23+
resolvedId: userId,
24+
warningType: "deprecated",
25+
};
1826
}
19-
20-
throw new Error('Either externalUserId or userId must be provided');
21-
};
27+
28+
throw new Error("Either externalUserId or userId must be provided");
29+
};

0 commit comments

Comments
 (0)