-
-
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
Support environment contentful parameter #4
Conversation
Hi @ryanhefner |
+1, this is very much needed for my project too. |
Thank @krambuhl! Sorry this took so long to get in. I hope this helps with your projects, @waelhasan and @diegopamio! Please let me know if you run into any other issues with this package, or |
Thank you! |
I'm having the issue you mentioned about contentful complaining with |
Hey @diegopamio, are you using |
Yes, I'm using
Is that an invalid combination? How could I achieve having the top query otherwise? Thanks! |
Ah, interesting, yeah that won't work since right now only the returned tree is being wrapped in the I suppose you could take all the data dependent stuff and move it into its own component, and pass the relevant |
Adds support for the
environment
contentful client parameter.Thanks for your effort on this package, it's making our next.js project much more straightforward. We appreciate your timeliness, since this update is to better support a client project. Let me know if I can do anything to make your life a little easier.