Skip to content
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

Updating grunge-stack to lastest versions #185

Open
jarimustonen opened this issue Feb 13, 2024 · 5 comments
Open

Updating grunge-stack to lastest versions #185

jarimustonen opened this issue Feb 13, 2024 · 5 comments

Comments

@jarimustonen
Copy link

Have you experienced this bug with the latest version of the template?

Yes

Steps to Reproduce

As the software evolves, the stack should be such that is could be updated to latest version. To this end, I suggest that grunge-stack is updated to latest versions.

I have tried this manually and was able to fix a few errors:

  1. A few packages must be added explicitly (@smithy/node-http-handler, @aws-sdk/lib-dynamodb, and @aws-sdk/client-dynamodb)
  2. isbot should be retained at the version 3.8.0 due to breaking changes in the version 4.0.0
  3. remix.config.js must be added the following configuration: browserNodeBuiltinsPolyfill: { modules: { querystring: true } }

Finally there is also an error related to cypress tests. The tests fail with:

Lambda init error: The requested module 'msw' does not provide an export named 'rest'
http://localhost:8811 timed out on retry 9 of 3, elapsed 95619ms, limit 90000ms
Error: Response code 500 (Internal Server Error)

I'm at lost here and really do not know what to do with this. If someone can help with this one, I could make pull request. Or maybe the Remix team would like to update the stack..

Expected Behavior

N/A

Actual Behavior

N/A

@McNouvion
Copy link

McNouvion commented Feb 16, 2024

I also ran in the cypress issue when it ran via the github action .... I got around it by updating mocks/index.js and replacing 'rest' with 'http'.

import { http } from "msw";
...  
http.post(`...

You will also need to update msw to the latest version. I have v2.1.5 right now ...

@gustavorino
Copy link

Any chance this project will be updated to use the vite version? is there anyone working on this?

@quantuminformation
Copy link

its a template, you are free to upgrade it yourself and sumbit a PR

@gustavorino
Copy link

its a template, you are free to upgrade it yourself and sumbit a PR

This is exactly why I asked, I am considering updating it, but won't do it if, lets say, someone else is already working on it.

@quantuminformation
Copy link

i think the latest v uses vite

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

No branches or pull requests

4 participants