Skip to content

Conversation

@TylerBarnes
Copy link
Contributor

@wardpeet I'm not sure why tests weren't failing before for your WP PQR PR but I think we need to revert it. It looks like it breaks builds for gatsby-source-wordpress w/ Gatsby v4. The int tests are currently failing (this PR fixes them).

Thinking about it a bit more that change is actually how it worked before I made gatsby-source-wordpress work w/ PQR. It has some caching logic to only run that ingest schema fn once every 10 seconds. The workers were failing since they didn't have the cached schema yet.

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Oct 19, 2021
@TylerBarnes TylerBarnes requested a review from wardpeet October 19, 2021 00:15
@wardpeet wardpeet merged commit 6cc964a into master Oct 19, 2021
@wardpeet wardpeet deleted the fix/wp-pqr branch October 19, 2021 05:35
@LekoArts LekoArts added topic: source-wordpress Related to Gatsby's integration with WordPress and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Oct 19, 2021
wardpeet pushed a commit to herecydev/gatsby that referenced this pull request Oct 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic: source-wordpress Related to Gatsby's integration with WordPress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants