Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add content/pt-br/stateless_apps.md #782

Merged
merged 10 commits into from
May 6, 2022

Conversation

MrErlison
Copy link
Collaborator

Add

  • content/pt-br/stateless_apps.md

@netlify
Copy link

netlify bot commented Apr 23, 2022

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit 8910188
🔍 Latest deploy log https://app.netlify.com/sites/cncfglossary/deploys/6274fccc2622720008cbefb1
😎 Deploy Preview https://deploy-preview-782--cncfglossary.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@edsoncelio edsoncelio added the lang/pt for Portuguese label Apr 23, 2022
Copy link
Collaborator

@jessicalins jessicalins left a comment

Choose a reason for hiding this comment

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

looking good, thanks @MrErlison !
I have the same question as in #781 (review) - but I really liked how you translated here ('Aplicações sem estado`) - so maybe we could go with this translation? cc @edsoncelio @brunoguidone

content/pt-br/stateless_apps.md Outdated Show resolved Hide resolved
content/pt-br/stateless_apps.md Outdated Show resolved Hide resolved
content/pt-br/stateless_apps.md Outdated Show resolved Hide resolved
@MrErlison
Copy link
Collaborator Author

looking good, thanks @MrErlison !
I have the same question as in #781 (review) - but I really liked how you translated here ('Aplicações sem estado`) - so maybe we could go with this translation? cc @edsoncelio @brunoguidone

It's ok for me. I will try to pay attention in the nexts PRs. Thank you.

@MrErlison MrErlison requested a review from jessicalins May 2, 2022 09:30
Copy link
Collaborator

@jessicalins jessicalins left a comment

Choose a reason for hiding this comment

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

just a few suggestions, then good to go from my side :)

content/pt-br/stateless_apps.md Outdated Show resolved Hide resolved
content/pt-br/stateless_apps.md Outdated Show resolved Hide resolved
MrErlison and others added 2 commits May 5, 2022 13:14
@MrErlison MrErlison requested a review from jessicalins May 5, 2022 16:15
Copy link
Collaborator

@jessicalins jessicalins left a comment

Choose a reason for hiding this comment

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

Hi @MrErlison , as a final consideration and after chatting with @brunoguidone @edsoncelio, we think it would be better to not directly translate stateless/stateful applications to aplicações sem/com estado. I think this would be one of those cases where we don't directly translate (for example, in this blog post this was also not translated)

I've added the suggested changes, after this would be good to go!

content/pt-br/stateless_apps.md Outdated Show resolved Hide resolved
content/pt-br/stateless_apps.md Outdated Show resolved Hide resolved
content/pt-br/stateless_apps.md Outdated Show resolved Hide resolved
@MrErlison MrErlison requested a review from jessicalins May 6, 2022 10:48
Copy link
Collaborator

@jessicalins jessicalins left a comment

Choose a reason for hiding this comment

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

LGTM

@edsoncelio edsoncelio merged commit 4d3579a into cncf:dev-pt May 6, 2022
@MrErlison MrErlison deleted the pt-br/stateless_apps branch May 6, 2022 12:41
seokho-son pushed a commit to seokho-son/glossary that referenced this pull request Mar 20, 2023
* Add content/pt-br/stateless_apps.md

* Update content/pt-br/stateless_apps.md

Co-authored-by: Jéssica Lins  <[email protected]>

* Update content/pt-br/stateless_apps.md

Co-authored-by: Jéssica Lins  <[email protected]>

* Update content/pt-br/stateless_apps.md

Co-authored-by: Jéssica Lins  <[email protected]>

* Update content/pt-br/stateless_apps.md

Co-authored-by: Jéssica Lins  <[email protected]>

* Update content/pt-br/stateless_apps.md

Co-authored-by: Jéssica Lins  <[email protected]>

* Update content/pt-br/stateless_apps.md

Co-authored-by: Jéssica Lins  <[email protected]>

* Update content/pt-br/stateless_apps.md

Co-authored-by: Jéssica Lins  <[email protected]>

* Update content/pt-br/stateless_apps.md

Co-authored-by: Jéssica Lins  <[email protected]>

* Update stateless_apps.md

Co-authored-by: Jéssica Lins  <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/pt for Portuguese
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants