-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
Fixes #1007, Add lowdb support #1038
Conversation
Deploy preview ready! Built with commit 0af5d71 |
Deploy preview ready! Built with commit 0af5d71 |
Deploy preview ready! Built with commit 0af5d71 |
I think putting this in its own package makes the most sense as there's an infinite number of ways JSON can be structured so trying to support those all in Also with each transformer type being its own package, they'll be easy to find via Google or the Gatsby website. Could you just add your questions here as a comment please. |
@KyleAMathews What's |
@gutenye they're useful conventions not a hard things-will-break-without-them requirement. They're there so that you can write queries following the transform chain. It's often useful for example to jump up from a transformed node to its file parent to get file specific information e.g. its size or file name, etc. I sometimes write queries too where I query for files and then jump to the right child. So it's strongly encouraged you add them. |
Looking good! FYI, there'll be some breaking changes for the API soonish see #858 Should be a pretty easy upgrade. |
@KyleAMathews Thanks for the heads up. |
I've a few questions and left notes in code.
Maybe we can extends builtin
gatsby-transfomer-json
with{"posts": [{"value": "a"}, ..]}
support instead of a new package.