-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Update Wasp Studio (deps, Express, tsx) #2848
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
base: main
Are you sure you want to change the base?
Conversation
@@ -2,7 +2,6 @@ | |||
<html lang="en"> | |||
<head> | |||
<meta charset="UTF-8" /> | |||
<link rel="icon" type="image/svg+xml" href="/vite.svg" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
vite.svg
didn't exist, so this just ended up being a 404 in the network tab
@@ -8,28 +8,25 @@ | |||
"main": "index.js", | |||
"keywords": [], | |||
"scripts": { | |||
"build": "npm run build:client && rm -rf dist && tsc && cp -r ./public ./dist/public", | |||
"build": "npm run build:client && rm -rf ./client/node_modules && rm -rf dist && tsc && cp -r ./public ./dist/public", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"@fastify/cors": "^8.3.0", | ||
"@fastify/static": "^6.11.2", | ||
"commander": "^11.0.0", | ||
"dotenv": "^16.0.3", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We didn't use .env
files so I've removed dotenv
as dep.
root: new URL("./public", import.meta.url).pathname, | ||
}); | ||
|
||
app.use(morgan("dev")); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Logging requests
This PR upgrade Wasp Studio:
ts-node
withtsx
(dev mode was broken, so I went withtsx
which has a watch mode so I can removenodemon
)nodemon
since it's no longer nededstudio/client/node_modules
to keep the binary size downAs a side-effect, all this fixed the issues with Wasp Studio not working in Firefox.
Closes #2774