-
Notifications
You must be signed in to change notification settings - Fork 378
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
Failed to launch dev server on Windows #52
Comments
Hi Matteo, thanks for sharing. Please detail what leads to this error, your environment, etc. to give more context. We didn't test under Windows yet 😊 |
Problem spotted, fix coming soon :) |
@MatteoGauthier we had a fix that we thought would resolve the problem but I just tested it and it looks like a new issues has started
any ideas how to fix this one @jeffsee55 ? |
@logan-anderson I still see double slashes in the path, are you using latest version? |
I did |
Latest version the CLI too? https://www.npmjs.com/package/tina-graphql-gateway-cli (This might be a dev-dependency) |
@jamespohalloran Yes tried on the latest version of that as well. |
I think I tracked down the issue. We are not handling windows when using the fast-glob package should have a fix in soon |
@MatteoGauthier This should be fixed now on the latest version. Let us know if you run into any more issues and thanks for letting us know about this one. |
On Windows as well and getting same error currently yarn run v1.22.10
$ yarn tina-gql server:start -c "next dev"
$ C:\code\practice\tina-cloud-starter\node_modules\.bin\tina-gql server:start -c "next dev"
Started Filesystem GraphQL server on port: 4001
Generating Tina config
Cannot find module 'C:\\code\\practice\\tina-cloud-starter\\.tina\\__generated__\\temp\\schema.js'
Require stack:
- C:\code\practice\tina-cloud-starter\node_modules\tina-graphql-gateway-cli\dist\index.js
- C:\code\practice\tina-cloud-starter\node_modules\tina-graphql-gateway-cli\bin\tina-gql, exiting...
C:\code\practice\tina-cloud-starter\node_modules\tina-graphql-gateway-cli\dist\index.js:1
Error: Cannot find module 'C:\\code\\practice\\tina-cloud-starter\\.tina\\__generated__\\temp\\schema.js'
Require stack:
- C:\code\practice\tina-cloud-starter\node_modules\tina-graphql-gateway-cli\dist\index.js
- C:\code\practice\tina-cloud-starter\node_modules\tina-graphql-gateway-cli\bin\tina-gql
at compile (C:\code\practice\tina-cloud-starter\node_modules\tina-graphql-gateway-cli\dist\index.js:276:22)
at async FSWatcher.<anonymous> (C:\code\practice\tina-cloud-starter\node_modules\tina-graphql-gateway-cli\dist\index.js:519:9) {
code: 'MODULE_NOT_FOUND',
requireStack: [
'C:\\code\\practice\\tina-cloud-starter\\node_modules\\tina-graphql-gateway-cli\\dist\\index.js',
'C:\\code\\practice\\tina-cloud-starter\\node_modules\\tina-graphql-gateway-cli\\bin\\tina-gql'
]
}
Done in 11.50s. |
@abrown2644 What version of the tina-graphql package are you using? |
@DirtyF I was missing that package actually. Just installed 0.1.12 but same error atm. |
@abrown2644 I believe the latest version is |
Had to bump to the following to get latest master to work,
However, it seems the path it's resolving to isn't correct for non homepage routes. Note the double Error: Unable to fetch, errors: |
@B-Stewart right should be |
Opened up a new ticket related to my duplicate path issue as it's kinda related to cross platform path handling, but needs to be fixed in the graphql package tinacms/tina-graphql-gateway#259 |
We just released a new version of the package @MatteoGauthier and @B-Stewart please upgrade and tell us if that fixes your issue. |
Bumping to
yarn allows me to choose the package to install instead of crashing
And the duplicate issue tinacms/tina-graphql-gateway#259 is still happening for me when I get it running again using yarn. |
This might be related. Just forked the repository on Windows and ran
|
@B-Stewart thanks for your report! There was an internal error with the way we where publishing our packages. This should be fixed if you update to the latest versions. I have made a PR that will force this #91. yarn upgrade-interactive and chose all the tina packages. You can also wait for the PR to me merged and then update your fork. @calvary-tucson that might be related as well. We will do some internal testing with a windows machine sometime this week. For now you could also try updating and see if the Issue persists. |
@logan-anderson The other problem I posted about might has something to do with the script command. I think it has something do to with the "-c" of the "yarn tina-gql server:start -c "next dev"" -- I know if I run the commands separately in different command prompts such as "yarn tina-gql server:start" in one and "yarn next dev" in another. Everything seems like it is working. Not sure if this helps. Tina Dependencies: |
I just forked from main that includes PR #91 and I'm getting this error on Windows unless I split the commands as in previous post :-
|
@ghowardMSD I think you're running into what @calvary-tucson mentioned, you just need to run those commands in separate windows: yarn tina-gql server:start And in another terminal: yarn next dev @calvary-tucson I think the issue you're seeing is unrelated to the terminal problem, we're likely just not building that path properly, we'll take a look. |
Yes, it does start ok when I run in separate windows and then I get the Pedro path issue. |
Poking at the command issue after someone ran into it while checking out Tina, I tried concurrently and it seemed to work:
|
@mitchmac The problem with concurrently is during CI, we wouldn't know when to shut down the GraphQL server. I believe James was playing around with this a while back but I don't think he got it all the way sorted out. |
@jeffsee55 Do we actually need to worry about CI right now for this, if we only changed the "dev" command? CI would be using build & export? |
@mitchmac Sure, it's just a pattern. They can also just run two processes. |
Related to tinacms/tinacms#1951 |
@mystrdat There was a couple issues with the navigation that have been fixed. Is your Fork up to date with the latest changes in the starter? |
Yes, I just cloned current main. If i If I run
|
@mystrdat Thanks for the details. Mind trying We changed the name of the command to start the server from |
I did try those, mentioned them above, that results are in the image I posted. It build but 404s on frontend with the SSR errors. |
@mystrdat are you still importing import { defineSchema } from '@tinacms/cli' |
|
Ok just wondering because To better debug this, can you run: yarn tinacms server:start And checkout `http://localhost:4001/altair. If you see a GraphQL GUI you should be able to run queries on it. That would at least tell us that the GraphQL portion is working. |
That was from when I tried |
Per @logan-anderson this should be ok but exposes issues deeper into use |
The deeper issue should be fixed now as well. Everything should work as expected on windows the same as unix |
Got this error
The text was updated successfully, but these errors were encountered: