8000 Spanish translation for section async by alvaropernas · Pull Request #1290 · fastapi/fastapi · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@alvaropernas
Copy link
Contributor

Hi,
Here is my first approach.
Appreciate any comments, thks!!

@codecov
Copy link
codecov bot commented Apr 19, 2020

Codecov Report

Merging #1290 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #1290   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          227       226    -1     
  Lines         6805      6785   -20     
=========================================
- Hits          6805      6785   -20     
Impacted Files Coverage Δ
fastapi/applications.py 100.00% <0.00%> (ø)
...orial/test_conditional_openapi/test_tutorial001.py

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c3545b6...c3545b6. Read the comment docs.

Copy link
Collaborator
@mariacamilagl mariacamilagl left a comment

Choose a reason for hiding this comment

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

Thank you so much for your effort @alvaropernas 🚀 👍

This was a long, complex one!

I know the changes may look a bit overwhelming, but they are minor details. Here's the summary of the suggestions:

  1. Removing double spaces and spaces after commas
  2. Maintaining consistency between "tu" and "usted"
  3. Keeping the central FastAPI terms like "path operation function" in English for consistency
  4. Other typos}

Thanks again! 🎉 🍰

alvaropernas and others added 24 commits April 27, 2020 00:17
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
alvaropernas and others added 21 commits April 27, 2020 00:42
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
Co-Authored-By: Camila Gutierrez <mariacamilagl@users.noreply.github.com>
@alvaropernas
Copy link
Contributor Author

Thanks Maria, I had doubts about what should be translated.
I will check the extra spaces and the use of "usted" for the next one.

@mariacamilagl
Copy link
Collaborator
mariacamilagl commented Apr 28, 2020

Thanks Maria, I had doubts about what should be translated.
I will check the extra spaces and the use of "usted" for the next one.

Hey, no worries. We are also always debating what should or shouldn't be translated. It isn't an exact science at all. Our main concern is to keep in English very technical terms that don´t have comfortable translations to Spanish, or that are core FastAPI terms.
But as you can see it's really subjective.

Thank you for your work! 🍰 🚀

Copy link
Collaborator
@mariacamilagl mariacamilagl left a comment

Choose a reason for hiding this comment

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

Thank you! 🚀

@tiangolo tiangolo merged commit 741de7f into fastapi:master May 16, 2020
@tiangolo
Copy link
Member

Thanks for your contribution @alvaropernas ! 🍰 🚀

And thanks @mariacamilagl for your review and help 🤓 ☕

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

0