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

A better intro/description to the usePreloadedQuery hook #3453

Closed
wants to merge 2 commits into from

Conversation

AdityaDonthy
Copy link
Contributor

@AdityaDonthy AdityaDonthy commented Apr 11, 2021

Instead of straight away asking the reader to go through the code and then understand what the hook does, the committed line gives a better intro of what the hook does. Also, what the inputs to the hook are and what it's output is. I feel like this important detail should be highlighted better. It gives a better context before reading the code

Instead of straight away asking the reader to go through the code and then understand what the hook does, the committed line gives a better intro of what the hook does. Also, what the inputs to the hook are and what it's output is. I feel like this important detail should be highlighted better.
@facebook-github-bot
Copy link
Contributor

@kassens has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@kassens merged this pull request in d8bd877.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants