-
Notifications
You must be signed in to change notification settings - Fork 8
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
getLoadContext
does not execute when passed to createRequestHandler
#94
Comments
@juanpprieto, hmmm, that could be a bug with context. You can update the
[build]
command = "npm run build:edge-demo"
publish = "packages/edge-demo-site/public"
+
+ [dev]
+ command = "npm run dev -w packages/edge-demo-site"
I'll update the dev command and netlify.toml for the demo site. As the remix dev command says, the dev server doesn't work for custom servers, i.e. not a node.js environment. |
If you make changes while debugging, you'll need to do a browser refresh if you make UI changes. For Edge function changes, the next request will pick up the latest changes. |
Just flagging this again as I'm also unable to do this, even with the steps described here. The |
@juanpprieto @jtaccinelli We're sorry you didn't get a prompt response here. I hope you were able to resolve your issue. Since this issue is quite stale, I'm going to close it, but please feel free to open a new issue with repro steps if you're still having issues, or (even better) seek help at https://answers.netlify.com/. Thanks. |
Hi @nickytonline hope you are well 🙏🏼
I been trying to add a context property in the edge template without any luck.
I checked a
loader
context's and it is missing the additionalfoo
property added above.I then tried debugging it via the
/packages/demo-edge-site
but the demo instance seems to be broken:The text was updated successfully, but these errors were encountered: