Skip to content

Hotfix/Embed: Theming not working#5976

Merged
kodiakhq[bot] merged 1 commit intomainfrom
hotfix/embed-theming
Dec 12, 2022
Merged

Hotfix/Embed: Theming not working#5976
kodiakhq[bot] merged 1 commit intomainfrom
hotfix/embed-theming

Conversation

@hariombalhara
Copy link
Copy Markdown
Member

@hariombalhara hariombalhara commented Dec 12, 2022

What does this PR do?

Fixes #5978

Embed UI Instruction shouldn't depend on plan now as everyone is at least PRO.

Environment: Production

Type of change

  • Bug fix (non-breaking change which fixes an issue)

@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 12, 2022

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

Name Status Preview Updated
cal ✅ Ready (Inspect) Visit Preview Dec 12, 2022 at 8:15AM (UTC)

@hariombalhara
Copy link
Copy Markdown
Member Author

There should be a test to verify that theme has been applied successfully. The test can simply check if light/dark classes are applied on document.

@hariombalhara hariombalhara requested a review from a team December 12, 2022 08:13
@hariombalhara hariombalhara changed the title Hotfix/Embed: UI Config shouldn't depend on plan now. Hotfix/Embed: Theming not working Dec 12, 2022
@hariombalhara hariombalhara added ♻️ autoupdate tells kodiak to keep this branch up-to-date automerge labels Dec 12, 2022
@hariombalhara hariombalhara self-assigned this Dec 12, 2022
@kodiakhq kodiakhq Bot merged commit 3080b47 into main Dec 12, 2022
@kodiakhq kodiakhq Bot deleted the hotfix/embed-theming branch December 12, 2022 08:26
@PeerRich PeerRich added the core area: core, team members only label Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge ♻️ autoupdate tells kodiak to keep this branch up-to-date core area: core, team members only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CAL-581] Embed UI instruction not working

3 participants