-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: support vite-dev-server middlewareMode #29219
base: develop
Are you sure you want to change the base?
feat: support vite-dev-server middlewareMode #29219
Conversation
|
@mucsi96 Thanks for the contribution! Are you able to add some tests around this behavior? |
@mucsi96 We'll be unable to merge this change without tests verifying the behavior. |
Hi! Sorry I got busy with work items. Let me try add a test |
…allow-extensibility-for-better-msw-support
@jennifer-shehane I have added an E2E test inspired by |
…allow-extensibility-for-better-msw-support
@mucsi96 Could you address the new conflicts with vite 5 support going in? |
Allow using vite dev server in middlerware mode as original does. See: https://vitejs.dev/config/server-options.html#server-middlewaremode
Helps resolving issues with MSW by allowing serving mock service worker from root.
PR Tasks
cypress-documentation
?type definitions
?