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

@lazaroalvarengalazaroalvarenga commented Feb 8, 2019

Copy link
Copy Markdown
Contributor

@glaucia86glaucia86 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

@glaucia86glaucia86 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
ContributorAuthor

@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

@cezaraugustocezaraugusto 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 threadcontent/docs/faq-ajax.md Outdated
Comment threadcontent/docs/faq-ajax.md Outdated
Comment threadcontent/docs/faq-ajax.md Outdated
@lazaroalvarenga

lazaroalvarenga commented Feb 8, 2019

Copy link
Copy Markdown
ContributorAuthor

@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

@cezaraugustocezaraugusto 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:masterFeb 8, 2019
@tesseralistesseralis mentioned this pull request Feb 8, 2019
Closed
90 tasks
@cezaraugustocezaraugusto 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-integrationBot 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-integrationBot 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 freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers
@glaucia86glaucia86glaucia86 approved these changes
@cezaraugustocezaraugustocezaraugusto approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

🌐 translatedTranslation 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