-
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
fix(gatsby-remark-images): support resolving markdown images from child nodes #28093
Conversation
any chance to get this reviewed? |
still waiting for feedback |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 from me
rebased on current master, hoping that the patch builds through now |
@KyleAMathews I am not sure what to do, the build-test failures do not appear to be related to my changes |
Anyone home? |
…ld nodes - in case where remark nodes are created from other nodes (e.g. gatsby-plugin-json-remark) the parent node of a remark document is not a file. - in order to resolve the images still, it is necessary to go up the parent chain until the closest File node
Tested the PR against @rburgst's reproduction and the image is now processed correctly: So this looks ready to go in assuming tests pass now with updated snapshots. |
Thanks again @rburgst for this PR and I'm very sorry it's taken so long to get merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As Kyle said, sorry this took so long to get to. Thanks for your contribution! 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚢
Failing tests were unrelated, merged! Thanks again! 🚀 |
(e.g. gatsby-plugin-json-remark) the parent node
of a remark document is not a file.
to go up the parent chain until the closest File node
Fixes #28094