Skip to content

fix(wistia-videos): set vite outDir to build [ES-372]#11025

Merged
Harika Kondur (harikakondur) merged 1 commit into
masterfrom
fix/wistia-vite-outdir-es-372
Jun 12, 2026
Merged

fix(wistia-videos): set vite outDir to build [ES-372]#11025
Harika Kondur (harikakondur) merged 1 commit into
masterfrom
fix/wistia-vite-outdir-es-372

Conversation

@harikakondur

@harikakondur Harika Kondur (harikakondur) commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • npm run deploy was failing with ENOENT: no such file or directory, scandir 'build' because Vite defaults to outputting to dist/ but the deploy script expected build/
  • Added build.outDir: 'build' to vite.config.ts to match the deploy script convention used by all other apps in this repo

…ES-372]

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 12, 2026 21:18
@harikakondur Harika Kondur (harikakondur) requested a review from a team as a code owner June 12, 2026 21:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes apps/wistia-videos deployment failures by aligning Vite’s build output directory with the npm run deploy script, which expects artifacts in ./build rather than Vite’s default dist/.

Changes:

  • Configure Vite to emit build artifacts to build/ via build.outDir in vite.config.ts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@harikakondur Harika Kondur (harikakondur) changed the title fix(wistia-videos): set vite outDir to build to match deploy script [ES-372] fix(wistia-videos): set vite outDir to build [ES-372] Jun 12, 2026
@harikakondur Harika Kondur (harikakondur) merged commit 522eb1f into master Jun 12, 2026
18 checks passed
@harikakondur Harika Kondur (harikakondur) deleted the fix/wistia-vite-outdir-es-372 branch June 12, 2026 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants