-
Notifications
You must be signed in to change notification settings - Fork 117
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
Deployment on Vercel #67
Comments
Also was able to deploy older revision here, from this branch |
Hi, I've no experience trying to deploy this to Vercel, but based on your successful deployment, does anything need to change in this repo to make the experience better? Or is there some docs we can add to the readme? |
Thanks for bringing this up, I have never tried to deploy on Vercel. I will see if I can reproduce this |
I figured out what was a problem, that was me 😄 I was overriding my config file with origin, going to PR proposal for it 👍 In the nutshell this is minor config tweak, I do not think there some Readme instructions or anything like this needed. But I'll doublecheck. |
Oh, actually giving it a second look I figured that the problem for me occurs due to:
Which I had commented out for my needs, but it still present issues if I uncomment it. Trying to reproduce it with the latest codebase and it is giving different errors now. Will spend a bit debugging. |
I figured out how to deploy, but not sure if this is an optimal way. Here is draft PR #75
PSPS: figured out the issue, it was use of process.env in I think at this point it works fine on Vercel, I might need to note it in README though. Also one outstanding issue, it is not building on Netlify preview, but it seems to be recurring behavior on other PRs as well. |
On the side note though, I built tracking page for own use. Would you be interested if I port it into this starter kit? |
Hi, I'm trying to deploy storefront-remix-starter on Vercel and unable to do so.
Here is the error I'm getting
I have customized
remix.config.js
as follows:The not found module is from this import:
import { createRoutesFromFolders } from '@remix-run/v1-route-convention';
It seems like more of Remix/Vercel issue than the starter, do not have enough experience with Remix to tell more.
Here is the repo code: https://github.com/dmi3y/storefront-remix-starter
And Vercel deploy: https://vercel.com/dmi3ii/storefront-remix-starter/5cZLctcchyQrhrpMFMq6X8RqmsUk#L58
The text was updated successfully, but these errors were encountered: