[Tracking]: gatsby-image PRs waiting on review and merge

[polarathene]

As these are starting to pile up and age, I created this issue to keep track of them.

I've tried reaching out several times for review to my PRs, and I've reviewed several others related to gatsby-image. @wardpeet is assigned to most, but seems to be rather busy lately with other work. I'd appreciate it if someone could spare the time to give these a look, especially #25325

Cheers.

Approved

• fix(gatsby-image): add required check for one of fluid or fixed #25371 - Do not crash if the image component is invoked without parameters
• fix(gatsby-transformer-sharp): removed unnecessary conversion to webp #25325 - fix(gatsby-transformer-sharp): removed unnecessary conversion to webp
• fix(gatsby-plugin-sharp): return correct height #25150 - fix(gatsby-plugin-sharp): return correct height

I've reviewed these and would like to seem them merged.

Update: @wardpeet has reviewed and approved each one, two have been left unmerged however, would be great if someone with authority could merge those!

Pending changes

• [gatsby-image] Propagate imgStyle prop down to non-js images #22607 - [gatsby-image] Propagate imgStyle prop down to non-js images

About a month since last activity, few days ago I reviewed and requested changes. If no activity after a while I'm happy to make the changes and recreate the PR, albeit waiting on core team about decision with how to approach ReactDOMServer. @pieh received no response since April 18.

Update: No feedback on this PR for 2 weeks since my activity there. I guess the ReactDOMServer approach isn't worth the time vs handling it with the v3 update where min React version can be raised.

• feat(gatsby-image): Add customProps as catchall #25283 - feat(gatsby-image): Add customProps as catchall

Update: @wardpeet decided against merging this. I still disagree with that decision and suggested workaround, but that discussion appears to have stalled.

Personal PRs awaiting review

• docs(gatsby-plugin-sharp): Base64 features #24999 - Docs(gatsby-plugin-sharp): Base64 features
• fix(gatsby-image): React hydration buster on image #24811 - fix(gatsby-image): React hydration buster on image

Several users are waiting on a fix for the hydration issue, while the base64 one is mostly docs so the feature can be actually discovered.

Update: Both have had an initial pass from @wardpeet but will need another review for changes, especially the hydration fix which added related fixes to art direction(I could separate these out into individual PRs if preferred).

[wardpeet]

Hey,

Thanks @polarathene, we have labels in place to make sure we catch all of these issues so this is a bit unnecessary.

Thanks for understanding!