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

Issue with redwood v7 #137

Open
ageddesi opened this issue Apr 7, 2024 · 6 comments
Open

Issue with redwood v7 #137

ageddesi opened this issue Apr 7, 2024 · 6 comments
Labels
bug Something isn't working windows Issues relating to windows machines working on it

Comments

@ageddesi
Copy link
Contributor

ageddesi commented Apr 7, 2024

I am trying to instal on redwood v7 and it successfully goes through the cli. But only the package.json file is changed. There is no demo page or any other stuff made.

@ageddesi
Copy link
Contributor Author

cmd: 'yarn add @redwoodjs-stripe/web',
stdout: '➤ YN0000: · Yarn 4.1.0\n' +
'➤ YN0000: ┌ Resolution step\n' +
'➤ YN0085: │ + @redwoodjs-stripe/web@npm:1.0.7\n' +
'➤ YN0000: └ Completed in 0s 295ms\n' +
'➤ YN0000: ┌ Post-resolution validation\n' +
"➤ YN0060: │ @redwoodjs/web is listed by your project with version 7.3.0, which doesn't satisfy what @redwoodjs-stripe/web (p0594b) requests (^5.4.3).\n" +
"➤ YN0060: │ memjs is listed by your project with version 1.3.1, which doesn't satisfy what @redwoodjs/api (p89477) requests (1.3.2).\n" +
"➤ YN0060: │ redis is listed by your project with version 4.6.13, which doesn't satisfy what @redwoodjs/api (p68c7e) requests (4.6.7).\n" +
"➤ YN0002: │ api@workspace:api doesn't provide @envelop/core (p29770), requested by @envelop/sentry.\n" +
"➤ YN0002: │ api@workspace:api doesn't provide graphql (peb1f8), requested by @envelop/sentry.\n" +
"➤ YN0002: │ api@workspace:api doesn't provide react (p30706), requested by @react-email/components.\n" +
"➤ YN0002: │ root-workspace-0b6124@workspace:. doesn't provide prettier (pfdc36), requested by prettier-plugin-tailwindcss.\n" +
"➤ YN0002: │ web@workspace:web doesn't provide webpack (p0988a), requested by postcss-loader.\n" +
'➤ YN0086: │ Some peer dependencies are incorrectly met; run yarn explain peer-requirements for details, where is the six-letter p-prefixed code.\n' +
'➤ YN0000: └ Completed\n' +
'➤ YN0000: ┌ Fetch step\n' +
'➤ YN0000: └ Completed in 1s 127ms\n' +
'➤ YN0000: ┌ Link step\n' +
'➤ YN0007: │ better-sqlite3@npm:8.6.0 must be built because it never has been before or the last one failed\n' +
"➤ YN0009: │ better-sqlite3@npm:8.6.0 couldn't be built successfully (exit code 1, logs can be found here: C:\Users\aaron\AppData\Local\Temp\xfs-f8bc799b\build.log)\n" +
'➤ YN0000: └ Completed in 2s 171ms\n' +
'➤ YN0000: · Failed with errors in 3s 751ms\n',
stderr: ''

@chrisvdm
Copy link
Owner

chrisvdm commented May 1, 2024

@ageddesi It's really strange, I haven't been able to reproduce the issue. I see you're on Windows - It might be Windows paths related. I'll do some more digging and back to you as soon as possible.

I'll also fix the peer dependency version range issue showing up in those logs. It wouldn't cause this issue but needs fixing regardless.

@chrisvdm chrisvdm added bug Something isn't working windows Issues relating to windows machines working on it labels May 1, 2024
@chrisvdm
Copy link
Owner

chrisvdm commented May 1, 2024

@ageddesi I would like to double check, are you running npx @redwoodjs-stripe/cli@latest setup? I see in those logs yarn add @redwoodjs-stripe/web but I'm not sure if you just ran that separately.

@chrisvdm
Copy link
Owner

chrisvdm commented May 1, 2024

I've tested it out on Windows with the cmd shell and I am able to find the files e.g web/src/pages/StripeDemoPage/StripeDemoPage.jsx

@ageddesi
Copy link
Contributor Author

ageddesi commented May 5, 2024

Hi @chrisvdm I can try repro on a fresh install of redwood. I am indeed on windows. I for now have manually added a stripe endpoint my self but the long term will be to use this package and help to make it better

@chrisvdm
Copy link
Owner

chrisvdm commented May 8, 2024

Hi @ageddesi, let me know if you manage to repro the issue on a fresh install.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working windows Issues relating to windows machines working on it
Projects
None yet
Development

No branches or pull requests

2 participants