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/service.md #810

Merged
merged 1 commit into from
Jul 31, 2022
Merged

Conversation

MrErlison
Copy link
Collaborator

Add content/pt-br/service.md

@netlify
Copy link

netlify bot commented Apr 30, 2022

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit 32e42fa
🔍 Latest deploy log https://app.netlify.com/sites/cncfglossary/deploys/626da8348eeb610008bc2cba
😎 Deploy Preview https://deploy-preview-810--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 30, 2022
@edsoncelio
Copy link
Collaborator

The definitions for technology and concepts need to contain three subheaders, but this definition doesn't contain this.
Do you know If it's right @seokho-son @jihoon-seo? This PR is to service.md definition.

@jihoon-seo
Copy link
Collaborator

@edsoncelio It seems that the term "Service" is categorized as "concept" but is okay without three subheaders, considering the existing content of this term.
(FYI - The last PR that updated the term "Service": #233)

Copy link
Collaborator

@edsoncelio edsoncelio 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!

Copy link
Collaborator

@brunoguidone brunoguidone 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 f9e866a into cncf:dev-pt Jul 31, 2022
@MrErlison MrErlison deleted the pt-br/service branch July 31, 2022 02:21
seokho-son pushed a commit to seokho-son/glossary that referenced this pull request Mar 20, 2023
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.

4 participants