Skip to content

Commit 618b772

Browse files
committed
debug generic
1 parent 0e21635 commit 618b772

File tree

7 files changed

+28
-20
lines changed

7 files changed

+28
-20
lines changed

.github/workflows/ci.yml

Lines changed: 14 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -296,22 +296,21 @@ jobs:
296296
strategy:
297297
fail-fast: false
298298
matrix:
299-
test-name:
300-
[
299+
test-name: [
301300
"generic",
302-
"express",
303-
"ap-flows",
304-
"localhost",
305-
"sessions",
306-
"sessions:staging",
307-
"handshake",
308-
"handshake:staging",
309-
"astro",
310-
"tanstack-react-start",
311-
"vue",
312-
"nuxt",
313-
"react-router",
314-
"custom",
301+
# "express",
302+
# "ap-flows",
303+
# "localhost",
304+
# "sessions",
305+
# "sessions:staging",
306+
# "handshake",
307+
# "handshake:staging",
308+
# "astro",
309+
# "tanstack-react-start",
310+
# "vue",
311+
# "nuxt",
312+
# "react-router",
313+
# "custom",
315314
]
316315
test-project: ["chrome"]
317316
include:

integration/templates/react-vite/src/main.tsx

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,9 @@ const Root = () => {
3535
routerPush={(to: string) => navigate(to)}
3636
routerReplace={(to: string) => navigate(to, { replace: true })}
3737
appearance={{
38+
colors: {
39+
colorPrimary: 'red',
40+
},
3841
layout: {
3942
showOptionalFields: true,
4043
},
@@ -45,6 +48,7 @@ const Root = () => {
4548
: undefined,
4649
}}
4750
>
51+
is this the correct app?
4852
<Outlet />
4953
</ClerkProvider>
5054
);

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@
4040
"test:integration:deployment:nextjs": "pnpm playwright test --config integration/playwright.deployments.config.ts",
4141
"test:integration:expo-web:disabled": "E2E_APP_ID=expo.expo-web pnpm test:integration:base --grep @expo-web",
4242
"test:integration:express": "E2E_DEBUG=1 E2E_DEBUG=1 E2E_APP_ID=express.* pnpm test:integration:base --grep @express",
43-
"test:integration:generic": "E2E_DEBUG=1 E2E_APP_ID=react.vite.*,next.appRouter.withEmailCodes* pnpm test:integration:base --grep @generic",
43+
"test:integration:generic": "E2E_DEBUG=1 E2E_APP_ID=react.vite.withEmailCodes pnpm test:integration:base --grep @generic",
4444
"test:integration:handshake": "DISABLE_WEB_SECURITY=true E2E_APP_1_ENV_KEY=sessions-prod-1 E2E_SESSIONS_APP_1_HOST=multiple-apps-e2e.clerk.app pnpm test:integration:base --grep @handshake",
4545
"test:integration:handshake:staging": "DISABLE_WEB_SECURITY=true E2E_APP_1_ENV_KEY=clerkstage-sessions-prod-1 E2E_SESSIONS_APP_1_HOST=clerkstage-sessions-prod-1-e2e.clerk.app pnpm test:integration:base --grep @handshake",
4646
"test:integration:localhost": "pnpm test:integration:base --grep @localhost",

packages/ui/src/Components.tsx

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,14 +17,13 @@ import type {
1717
UserProfileProps,
1818
WaitlistProps,
1919
} from '@clerk/shared/types';
20-
21-
import type { Appearance } from './internal/appearance';
2220
import { createDeferredPromise } from '@clerk/shared/utils';
2321
import React, { Suspense } from 'react';
2422

2523
import type { AppearanceCascade } from './customizables/parseAppearance';
2624
// NOTE: Using `./hooks` instead of `./hooks/useClerkModalStateParams` will increase the bundle size
2725
import { useClerkModalStateParams } from './hooks/useClerkModalStateParams';
26+
import type { Appearance } from './internal/appearance';
2827
import type { ClerkComponentName } from './lazyModules/components';
2928
import {
3029
BlankCaptchaModal,
@@ -293,6 +292,10 @@ const Components = (props: ComponentsProps) => {
293292
impersonationFab: false,
294293
});
295294

295+
console.log('props.options.appearance', JSON.stringify(props.options.appearance));
296+
console.log('props.options', JSON.stringify(props.options));
297+
console.log('state.appearance', JSON.stringify(state.appearance));
298+
296299
const {
297300
googleOneTapModal,
298301
signInModal,

packages/ui/src/components/SignUp/SignUpStart.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -439,7 +439,6 @@ function SignUpStartInternal(): JSX.Element {
439439
legalAccepted={Boolean(formState.legalAccepted.checked) || undefined}
440440
/>
441441
)}
442-
hello there
443442
{shouldShowForm && (
444443
<SignUpForm
445444
handleSubmit={handleSubmit}

packages/ui/src/customizables/AppearanceContext.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ const AppearanceProvider = (props: AppearanceProviderProps) => {
1515
const value = parseAppearance(props);
1616

1717
return { value };
18-
}, [props.appearance, props.globalAppearance]);
18+
}, [props.appearance, props.globalAppearance, props.appearanceKey]);
1919

2020
return <AppearanceContext.Provider value={ctxValue}>{props.children}</AppearanceContext.Provider>;
2121
};

packages/ui/src/customizables/parseAppearance.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,9 @@ const defaultCaptchaOptions: ParsedCaptcha = {
6868
*/
6969
export const parseAppearance = (cascade: AppearanceCascade): ParsedAppearance => {
7070
const { globalAppearance, appearance: componentAppearance, appearanceKey } = cascade;
71+
console.log('globalAppearance', globalAppearance);
72+
console.log('componentAppearance', componentAppearance);
73+
console.log('appearanceKey', appearanceKey);
7174

7275
const appearanceList: Appearance[] = [];
7376
[globalAppearance, globalAppearance?.[appearanceKey as PublicAppearanceTopLevelKey], componentAppearance].forEach(a =>

0 commit comments

Comments
 (0)