Skip to content

Translation of "Hooks at a Glance" page#70

Merged
carburo merged 6 commits into
reactjs:masterfrom
GiuMagnani:patch-2
Feb 8, 2019
Merged

Translation of "Hooks at a Glance" page#70
carburo merged 6 commits into
reactjs:masterfrom
GiuMagnani:patch-2

Conversation

@GiuMagnani

@GiuMagnani GiuMagnani commented Feb 4, 2019

Copy link
Copy Markdown
Contributor

@alejandronanez alejandronanez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey @GiuMagnani, thanks for your hard work here, this is getting super close to what we need! 🙌

I left a couple of suggestions that should be addressed before merging this!

Thanks a lot!

Comment thread content/docs/hooks-overview.md Outdated
Comment thread content/docs/hooks-overview.md Outdated
alejandronanez and others added 2 commits February 4, 2019 22:44
Fix missed "Hooks"

Co-Authored-By: GiuMagnani <glmagnani@gmail.com>
Fixed wrong translation for "Effect Hook".

Co-Authored-By: GiuMagnani <glmagnani@gmail.com>
@GiuMagnani

Copy link
Copy Markdown
Contributor Author

Thanks @alejandronanez for the corrections! I already committed them. Let me know if there is anything else.

@alejandronanez alejandronanez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Awesome work as always @GiuMagnani!

Thank you very much!

@alejandronanez alejandronanez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@GiuMagnani, oops, it seems that you have some merge conflicts on your branch.

Please, fix them and this will be ready to merge!

@GiuMagnani

Copy link
Copy Markdown
Contributor Author

@alejandronanez I'll take a look, thanks!

@carburo carburo merged commit b726491 into reactjs:master Feb 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants