Skip to content

Translation of the "AJAX and APIs" page - #9

Merged
cezaraugusto merged 3 commits into
reactjs:masterfrom
lazaroalvarenga:faq-ajax
Feb 8, 2019
Merged

Translation of the "AJAX and APIs" page#9
cezaraugusto merged 3 commits into
reactjs:masterfrom
lazaroalvarenga:faq-ajax

Conversation

@lazaroalvarenga

@lazaroalvarenga lazaroalvarenga commented Feb 8, 2019

Copy link
Copy Markdown
Contributor

#1

@glaucia86 glaucia86 added the needs review A pull request ready to be reviewed label Feb 8, 2019
glaucia86
glaucia86 approved these changes Feb 8, 2019
View reviewed changes

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

LGTM 馃憤

@glaucia86

glaucia86 commented Feb 8, 2019

Copy link
Copy Markdown
Contributor

cc: @cezaraugusto @WendellAdriel can see this PR?! For me looks good to me! :)

@lazaroalvarenga

lazaroalvarenga commented Feb 8, 2019

Copy link
Copy Markdown
Contributor Author

@glaucia86 I updated with changes to standardize the translation of the word call. It's fine?

@WendellAdriel

WendellAdriel commented Feb 8, 2019

Copy link
Copy Markdown
Contributor

@glaucia86 @cezaraugusto @lazaroalvarenga
I really think that we shouldn't translate code blocks, I put that as the first item on #4

cezaraugusto
cezaraugusto suggested changes Feb 8, 2019
View reviewed changes

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

thanks for doing this @lazaroalvarenga. please see comments

Comment thread content/docs/faq-ajax.md Outdated
Comment thread content/docs/faq-ajax.md Outdated
Comment thread content/docs/faq-ajax.md Outdated
@lazaroalvarenga

lazaroalvarenga commented Feb 8, 2019

Copy link
Copy Markdown
Contributor Author

@glaucia86 @cezaraugusto @lazaroalvarenga
I really think that we shouldn't translate code blocks, I put that as the first item on #4

ok! Sorry for that. I'll unmake the code blocks translation.

@glaucia86

glaucia86 commented Feb 8, 2019

Copy link
Copy Markdown
Contributor

@lazaroalvarenga todos os acertos que vc for fazendo de acordo com review marque como 'Resolve Conversation' ;)

@glaucia86

glaucia86 commented Feb 8, 2019

Copy link
Copy Markdown
Contributor

cc: @cezaraugusto @lazaroalvarenga already did the changes here. I think that we can approve his request. What do u think?

cezaraugusto
cezaraugusto approved these changes Feb 8, 2019
View reviewed changes

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

@lazaroalvarenga way to go, thanks!

@cezaraugusto
cezaraugusto merged commit 3eb79dd into reactjs:master Feb 8, 2019
@tesseralis tesseralis mentioned this pull request Feb 8, 2019
Closed
90 tasks
@cezaraugusto cezaraugusto added 馃寪 translated Translation is completed for this page and removed needs review A pull request ready to be reviewed labels Feb 8, 2019
fjoshuajr pushed a commit that referenced this pull request Feb 10, 2019
@tibuurcio @fjoshuajr
Fix some link translations in initial PR's (#58)
74fca93 
* Fix links in lists-and-keys page #7

* Fix links in faq-ajax page #9

* Fix links in faq-build page #12

* Fix links in web-components page #27
devin-ai-integration Bot pushed a commit to WalissonRodrigo/pt-br.react.dev that referenced this pull request Jul 9, 2026
@lazaroalvarenga @cezaraugusto
Translation of the "AJAX and APIs" page (reactjs#9)
a2f2431
devin-ai-integration Bot pushed a commit to WalissonRodrigo/pt-br.react.dev that referenced this pull request Jul 9, 2026
@tibuurcio @fjoshuajr
Fix some link translations in initial PR's (reactjs#58)
b432f7b 
* Fix links in lists-and-keys page reactjs#7

* Fix links in faq-ajax page reactjs#9

* Fix links in faq-build page reactjs#12

* Fix links in web-components page reactjs#27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers
@glaucia86 glaucia86 glaucia86 approved these changes
@cezaraugusto cezaraugusto cezaraugusto approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

馃寪 translated Translation is completed for this page

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@lazaroalvarenga @glaucia86 @WendellAdriel @cezaraugusto