Skip to content

Commit

Permalink
Get rid of themeBasis warning in test suite
Browse files Browse the repository at this point in the history
  • Loading branch information
keithwillcode committed May 25, 2024
1 parent ff1e888 commit 9c994b7
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
6 changes: 5 additions & 1 deletion apps/web/lib/app-providers-app-dir.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,11 @@ function getThemeProviderProps(props: {

const isBookingPageThemeSupportRequired = themeSupport === ThemeSupport.Booking;

if ((isBookingPageThemeSupportRequired || props.isEmbedMode) && !props.themeBasis) {
if (
!process.env.NEXT_PUBLIC_IS_E2E &&
(isBookingPageThemeSupportRequired || props.isEmbedMode) &&
!props.themeBasis
) {
console.warn(
"`themeBasis` is required for booking page theme support. Not providing it will cause theme flicker."
);
Expand Down
2 changes: 1 addition & 1 deletion apps/web/lib/app-providers.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,7 @@ function getThemeProviderProps({
const isBookingPageThemeSupportRequired = themeSupport === ThemeSupport.Booking;
const themeBasis = props.themeBasis;

if ((isBookingPageThemeSupportRequired || isEmbedMode) && !themeBasis) {
if (!process.env.NEXT_PUBLIC_IS_E2E && (isBookingPageThemeSupportRequired || isEmbedMode) && !themeBasis) {
console.warn(
"`themeBasis` is required for booking page theme support. Not providing it will cause theme flicker."
);
Expand Down

0 comments on commit 9c994b7

Please sign in to comment.