Skip to content

Translate Higher Order Components#125

Merged
carburo merged 7 commits into
reactjs:masterfrom
elyalvarado:higher-order-components
Feb 10, 2019
Merged

Translate Higher Order Components#125
carburo merged 7 commits into
reactjs:masterfrom
elyalvarado:higher-order-components

Conversation

@elyalvarado

Copy link
Copy Markdown
Contributor

No description provided.

@elyalvarado

Copy link
Copy Markdown
Contributor Author

#4

@icarlossz icarlossz 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.

Good job! I left you some comments! 👍

Comment thread content/docs/higher-order-components.md
Comment thread content/docs/higher-order-components.md Outdated
Comment thread content/docs/higher-order-components.md Outdated
Comment thread content/docs/higher-order-components.md Outdated
Comment thread content/docs/higher-order-components.md Outdated
Comment thread content/docs/higher-order-components.md
Comment thread content/docs/higher-order-components.md Outdated
@elyalvarado

Copy link
Copy Markdown
Contributor Author

Thanks @icarlossz for your reviews, I merged some and left comments on the others

@icarlossz

Copy link
Copy Markdown
Contributor

Thanks! I learned a lot in this PR 😅

@elyalvarado

Copy link
Copy Markdown
Contributor Author

Hey @RamirezAlex, @carburo will you take a look at this translation? I appreciate your comments from previous translations and I think this one will benefit from an extra pair of eyes.

@RamirezAlex RamirezAlex 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.

Good work in general, I only that small type and the missing comma :)

Comment thread content/docs/higher-order-components.md Outdated
Comment thread content/docs/higher-order-components.md Outdated
@elyalvarado

Copy link
Copy Markdown
Contributor Author

@carburo @alejandronanez I think this is ready for merging

@carburo carburo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks good! I mostly fixed a few typos and added some missing inverted exclamation and question marks.
Merging...

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

Labels

None yet

4 participants