Skip to content

Conversation

@Simply007
Copy link
Contributor

@Simply007 Simply007 commented Jan 7, 2021

Description

The testPluginOptionsSchema returns Promise so that calling it should be in async await construct for test code sample.

Related Issues

Found during implementation kontent-ai/gatsby-packages#167

The testPluginOptionsSchema returns promise.
@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Jan 7, 2021
@Simply007 Simply007 changed the title Update plugin option schema validation code samples Update plugin option schema validation code sample Jan 7, 2021
@LekoArts LekoArts added type: documentation An issue or pull request for improving or updating Gatsby's documentation and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Jan 8, 2021
@Simply007 Simply007 marked this pull request as ready for review January 8, 2021 09:44
@Simply007
Copy link
Contributor Author

@LekoArts - is there anything else to do on my side?

Copy link
Contributor

@laurieontech laurieontech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Absolutely correct. When we changed this function I must have missed these docs. Thanks for addressing!

@laurieontech laurieontech added the bot: merge on green Gatsbot will merge these PRs automatically when all tests passes label Jan 21, 2021
@gatsbybot gatsbybot merged commit 9dce586 into gatsbyjs:master Jan 21, 2021
@Simply007 Simply007 deleted the patch-2 branch January 21, 2021 14:26
pragmaticpat pushed a commit to pragmaticpat/gatsby that referenced this pull request Apr 28, 2022
* Update configuring-usage-with-plugin-options.md

The testPluginOptionsSchema returns promise.

* Add mention about Kontent implementation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot: merge on green Gatsbot will merge these PRs automatically when all tests passes type: documentation An issue or pull request for improving or updating Gatsby's documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants