Thanks to visit codestin.com
Credit goes to github.com

Skip to content

🌐 Update Portuguese Translation for docs/pt/docs/deployment/https.md#13317

Merged
alejsdev merged 4 commits into
fastapi:masterfrom
Joao-Pedro-P-Holanda:docs/update-https
Feb 7, 2025
Merged

🌐 Update Portuguese Translation for docs/pt/docs/deployment/https.md#13317
alejsdev merged 4 commits into
fastapi:masterfrom
Joao-Pedro-P-Holanda:docs/update-https

Conversation

@Joao-Pedro-P-Holanda

Copy link
Copy Markdown
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

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

Really great @Joao-Pedro-P-Holanda 😎

Just one silly suggestion :D

Comment thread docs/pt/docs/deployment/https.md Outdated
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
Copy Markdown
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 🤓

Comment thread docs/pt/docs/deployment/https.md Outdated
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
Copy Markdown
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

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

🚀

@github-actions

github-actions Bot commented Feb 7, 2025

Copy link
Copy Markdown
Contributor

@alejsdev alejsdev merged commit 52c1488 into fastapi:master Feb 7, 2025
@alejsdev

alejsdev commented Feb 7, 2025

Copy link
Copy Markdown
Member

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