kinngh / shopify-node-express-mongodb-app

An embedded Shopify app starter template made with Node, Express, React and Vite, with all the required stuff hooked up.
https://x.com/kinngh
MIT License
289 stars 98 forks source link

Getting a 404 error #1

Closed waleakande closed 2 years ago

waleakande commented 2 years ago

I followed your instructions. When I try to install the app from the Partners dashboard it redirects me to a 404 page.

waleakande commented 2 years ago

I wanted to give a quick update, when I try to install the app it attempts to take me through the installation process. It then times out after 3mins. It returning a 504 error now.

kinngh commented 2 years ago

Can I have more info into what's your setup like? Like Localtunnel, Mongo and other configuration? Because I've been working on this template as a starter for 2 production grade apps and the boilerplate works out of the box, so I'm pretty sure it's just a configuration issue and the documentation needs to be clearer on setting up the repo

waleakande commented 2 years ago

I figure it out my configuration was off earlier.

kinngh commented 2 years ago

Glad you figured it out! Please feel free to open any issues if they pop up!

ivorpad commented 2 years ago

@waleakande What was off in your config? I keep getting the same 502.

@kinngh Why are you exporting createServer? I suggest using named exports.

When running npm run dev the server is not starting (createServer never gets called)

kinngh commented 2 years ago

@kinngh Why are you exporting createServer? I suggest using named exports.

When running npm run dev the server is not starting (createServer never gets called)

@ivorpad The implementation is half done. Currently it runs in production mode only, I had some issues with dev mode so that's still pending. The issue is on my end because I should have made it clear that dev mode with HMR is currently under testing.

To get the repo running, first run npm run build and then npm run start

kinngh commented 2 years ago

Ref comments on commit 71dcc47