Skip to content
This repository was archived by the owner on Oct 29, 2025. It is now read-only.

Conversation

@hariombalhara
Copy link
Member

@hariombalhara hariombalhara commented Jun 10, 2024

Fixes #360

Possible regression from #359. As we stopped using turbo for build, embed-core dependency of @calcom/web stopped building causing the public/embed folder to be not generated.

@hariombalhara hariombalhara marked this pull request as ready for review June 10, 2024 09:07
@hariombalhara hariombalhara changed the title Fix missing embed folder in public Fix missing embed folder in web/public Jun 10, 2024
RUN yarn --cwd packages/prisma seed-app-store
# Build and make embed servable from web/public/embed folder
RUN yarn --cwd packages/embeds/embed-core workspace @calcom/embed-core run build
RUN yarn --cwd apps/web workspace @calcom/web run build
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note: Docker build was failing for me at @calcom/web run build command(due to type check) and it is unrelated to this change. I got the build passing through by adding CI=1 env variable for line 37.

@keithwillcode keithwillcode merged commit 7218b23 into calcom:main Jul 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

domain/embed/embed.js returns 404 error

3 participants