Skip to content

docs(hooks): improving useImperativeHandle example #542

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

Merged
merged 4 commits into from
Sep 4, 2022

Conversation

mateoguzmana
Copy link
Contributor

Just stumbled upon the current explanation for the useImperativeHandle and saw that it is pretty basic.

I found a better way to type useImperativeHandle on this Stackoverflow answer and thought we can use this one instead, which is more complete.

@filiptammergard
Copy link
Collaborator

Thanks @mateoguzmana, I think this is a great improvement. Can you run yarn gen-readme to get the same changes to README.md? Then we're good to go!

@swyxio
Copy link
Collaborator

swyxio commented Sep 4, 2022

@filiptammergard maybe we should make a github action to do that for us so that we dont have to remind contributors

@filiptammergard
Copy link
Collaborator

@filiptammergard maybe we should make a github action to do that for us so that we dont have to remind contributors

Yeah I agree. I'll take a look at it!

@mateoguzmana
Copy link
Contributor Author

Done @filiptammergard. Thanks!

@filiptammergard
Copy link
Collaborator

Done @filiptammergard. Thanks!

Awesome, thanks @mateoguzmana!

@filiptammergard filiptammergard merged commit 2859208 into typescript-cheatsheets:main Sep 4, 2022
@mateoguzmana mateoguzmana deleted the patch-1 branch December 23, 2022 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants