search

GrappleGQL / gatsby-source-wagtail

Plugin for sourcing Gatsby data from Wagtail CMS
17 stars 12 forks source link

useStaticQuery broken #10

Closed fabienheureux closed 4 years ago

fabienheureux commented 4 years ago

Following Slack's dm with @NathHorrigan, gatsby-source-wagtail breaks useStaticQuery hook. The fix can be pulled from https://github.com/birkir/gatsby-source-graphql-universal/commit/a68e20d0cad230a778928809300959a95a536788#diff-6ad5ff1468daded9b3e80d9db6782084

Will submit a PR later today / next week trying to fix this.

fabienheureux commented 4 years ago

Fixed by https://github.com/NathHorrigan/gatsby-source-wagtail/pull/11

jimbofreedman commented 4 years ago

@fabienheureux I'm using your PR but am still seeing useStaticQuery errors on the starter site. Is there any other dependency you needed to change?

tbrlpld commented 4 years ago

Same issue here. I still get the same error although I have installed the package with the merge commit as the version.

fabienheureux commented 4 years ago

This PR need a bit of work actually. Did you have any other errors ? (murmurhash related, for example)

NathHorrigan commented 4 years ago

I’ve actually got this fixed in another branch! On 30 Jun 2020, 13:01 +0100, fabienheureux notifications@github.com, wrote:

This PR need a bit of work actually. Did you have any other errors ? (murmurhash related, for example) — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

NathHorrigan commented 4 years ago

@fabienheureux @tbrlpld @jimbofreedman

There will be a new NPM release in the next day which will fix useStaticQuery and allow Gatsby to download Wagtail images and process them via Sharp/Gatsby Image.

tbrlpld commented 4 years ago

Just tried it out. Works like a charm 👏

Thank you very much for this update.

NathHorrigan commented 4 years ago

No Worries. Closing.