feat(styling and theme): Added Parameter for Styling and Theme

[gaokevin1]

Related Issues

Description

This PR adds the ability to control styling and theme from a URL query parameter in the Auth Hosting application.

Screenshots

Must

• [ ] 📱 Responsiveness (mobile/XL resolutions)
• [ ] 🧪 Tests
• [ ] 📃 Documentation (if applicable)

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
auth-hosting	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 3, 2024 4:17pm

[vercel[bot]]

Deployment failed with the following error:

You don't have permission to create a Preview Deployment for this project.

View Documentation: https://vercel.com/docs/accounts/team-members-and-roles

[gaokevin1]

Also @asafshen, do you want another test for this PR besides this one?

test('that the flow can be customized with search params', async () => {
		window.location.pathname = `/${packageJson.homepage}/${validProjectId}`;
		window.location.search = `?debug=${debug}&flow=${flowId}`;
		render(<App />);
		expect(mockDescope).toHaveBeenCalledWith(
			expect.objectContaining({ debug, flowId })
		);
	});

[asafshen]

from what I see in the PR in descope-js this was not added yet to react-sdk https://github.com/descope/descope-js/pull/744/files#diff-b24066d0d30844fccfe74b02e0ee2dedf42a1cf7fa1bd6377fe6fd08695fd234 only to web-component at this stage

[gaokevin1]

Yeah we'll hold off until it's added to the React SDK.

[gaokevin1]

@orius123 can you please review this now?

[orius123]

@gaokevin1 build is failing

[gaokevin1]

@orius123 can you please review again? Build should be passing now.

[gaokevin1]

That's because of my vercel access not because of the build. If you push something you'll see. @asafshen am I good to merge?

[dorsha]

This was not added to the dynamicSdk, which means it is not available to customers. So we have a doc that was not tested in production.

[gshriki]

interesting. I was sure @anvibanga checked it...

[anvibanga]

@chris4490 @gaokevin1

[chris4490]

Yea I thought I'd checked it as well.