Skip to content
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

docs(creating-and-modifying-pages): remove unnecessary promise #11697

Merged
merged 2 commits into from
Feb 12, 2019

Conversation

jkjustjoshing
Copy link
Contributor

@jkjustjoshing jkjustjoshing commented Feb 11, 2019

I also added a "note" indicating that you can still use this function asynchronously. Instead of (or in addition to) this, we might want to add a true async example.

I also added a "note" indicating that you can still use this function asynchronous.
@jkjustjoshing jkjustjoshing requested a review from a team February 11, 2019 21:21
@jkjustjoshing
Copy link
Contributor Author

Change made after this Twitter exchange https://twitter.com/gatsbyjs/status/1095068494075326465

Copy link
Contributor

@m-allanson m-allanson left a comment

Choose a reason for hiding this comment

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

Thanks @jkjustjoshing 👍

@m-allanson
Copy link
Contributor

m-allanson commented Feb 12, 2019

Oops, the linting failed - I'll update this quickly.

@wardpeet wardpeet added the type: documentation An issue or pull request for improving or updating Gatsby's documentation label Feb 12, 2019
@wardpeet wardpeet changed the title Remove unnecessary promise wrapper in example docs(creating-and-modifying-pages): remove unnecessary promise Feb 12, 2019
@m-allanson m-allanson merged commit df2506c into master Feb 12, 2019
@m-allanson m-allanson deleted the jkjustjoshing-patch-1 branch February 12, 2019 12:27
gurpreet-hanjra pushed a commit to gurpreet-hanjra/gatsby that referenced this pull request Feb 14, 2019
…yjs#11697)

* Remove unnecessary promise wrapper in example

I also added a "note" indicating that you can still use this function asynchronous.

* Lint
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

None yet

3 participants