epicweb-dev / epic-stack

This is a Full Stack app starter with the foundational things setup and configured for you to hit the ground running on your next EPIC idea.
https://www.epicweb.dev/epic-stack
MIT License
4.32k stars 355 forks source link

update all dependencies #749

Closed kentcdodds closed 3 months ago

kentcdodds commented 3 months ago

Had an issue with e2e tests during the onboarding test:

Error: Dynamic require of "stream" is not supported
    at file:///Users/kentcdodds/code/epicweb-dev/epic-stack/node_modules/@react-email/render/dist/index.mjs:22:9
    at file:///Users/kentcdodds/code/epicweb-dev/epic-stack/node_modules/@react-email/render/dist/index.mjs:104:26
    at Generator.next (<anonymous>)
    at file:///Users/kentcdodds/code/epicweb-dev/epic-stack/node_modules/@react-email/render/dist/index.mjs:41:61
    at new Promise (<anonymous>)
    at __async (file:///Users/kentcdodds/code/epicweb-dev/epic-stack/node_modules/@react-email/render/dist/index.mjs:25:10)
    at readStream (file:///Users/kentcdodds/code/epicweb-dev/epic-stack/node_modules/@react-email/render/dist/index.mjs:94:30)
    at file:///Users/kentcdodds/code/epicweb-dev/epic-stack/node_modules/@react-email/render/dist/index.mjs:133:26
    at Generator.next (<anonymous>)
    at file:///Users/kentcdodds/code/epicweb-dev/epic-stack/node_modules/@react-email/render/dist/index.mjs:41:61

Not sure what's going on with this one.

Also got this error:

PrismaClientKnownRequestError: 
Invalid `prisma.note.upsert()` invocation:

Foreign key constraint failed on the field: `foreign key`
    at In.handleRequestError (/Users/kentcdodds/code/epicweb-dev/epic-stack/node_modules/@prisma/client/runtime/library.js:122:6877)
    at In.handleAndLogRequestError (/Users/kentcdodds/code/epicweb-dev/epic-stack/node_modules/@prisma/client/runtime/library.js:122:6211)
    at In.request (/Users/kentcdodds/code/epicweb-dev/epic-stack/node_modules/@prisma/client/runtime/library.js:122:5919)
    at l (/Users/kentcdodds/code/epicweb-dev/epic-stack/node_modules/@prisma/client/runtime/library.js:127:11167)
    at action (file:///Users/kentcdodds/code/epicweb-dev/epic-stack/app/routes/users+/$username_+/__note-editor.server.tsx:105:22)
    at Object.callRouteAction (/Users/kentcdodds/code/epicweb-dev/epic-stack/node_modules/@remix-run/server-runtime/dist/data.js:37:16)
    at /Users/kentcdodds/code/epicweb-dev/epic-stack/node_modules/@remix-run/router/router.ts:4266:21
    at callLoaderOrAction (/Users/kentcdodds/code/epicweb-dev/epic-stack/node_modules/@remix-run/router/router.ts:4328:16)
    at async Promise.all (index 2)
    at callDataStrategyImpl (/Users/kentcdodds/code/epicweb-dev/epic-stack/node_modules/@remix-run/router/router.ts:4169:17)

That one seemed to happen during a note edit test. Gotta look into these failures before merging.

kentcdodds commented 3 months ago

The first issue has been reported here: https://github.com/resend/react-email/issues/1486

The second issue was a flakey test that I resolved :)