Skip to content

Conversation

Joao-Pedro-P-Holanda
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the lang-all Translations label Feb 3, 2025
@alejsdev alejsdev added lang-pt Portuguese translations awaiting-review labels Feb 3, 2025
Copy link
Contributor

@ceb10n ceb10n left a comment

Choose a reason for hiding this comment

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

Really great @Joao-Pedro-P-Holanda 😎

Just one silly suggestion :D


O processo de aquisição de um desses certificados costumava ser complicado, exigia bastante papelada e os certificados eram bastante caros.

Mas então <a href="https://letsencrypt.org/" class="external-link" target="_blank">Let's Encrypt</a> foi criado.
Mas então **<a href="https://letsencrypt.org/" class="external-link" target="_blank">Let's Encrypt</a>** foi criado.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it sounds a little strange without the article.

I prefer to translate as "Mas então o Let's Encrypt..."

But it's not wrong, it's just a suggestion 🤓

Mas uma vez que você saiba o básico de **HTTPS para desenvolvedores**, você pode combinar e configurar diferentes ferramentas facilmente para gerenciar tudo de uma forma simples.

Em alguns dos próximos capítulos, eu mostrarei para você vários exemplos concretos de como configurar o **HTTPS** para aplicações **FastAPI**.
🔒
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you accidentally hit enter here :D

emoji

adding article before Let's encrypt on line 52

moving emoji from line 200 to 199
Copy link
Contributor

@ceb10n ceb10n left a comment

Choose a reason for hiding this comment

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

🚀

Copy link
Contributor

github-actions bot commented Feb 7, 2025

@alejsdev alejsdev merged commit 52c1488 into fastapi:master Feb 7, 2025
52 checks passed
@alejsdev
Copy link
Member

alejsdev commented Feb 7, 2025

Thanks for the update @Joao-Pedro-P-Holanda 🤓
And thanks for the review @ceb10n 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved-2 lang-all Translations lang-pt Portuguese translations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants