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 service models and modify documentation #1270

Merged
merged 4 commits into from
Sep 13, 2022
Merged

Add service models and modify documentation #1270

merged 4 commits into from
Sep 13, 2022

Conversation

KiranSatyaRaj
Copy link
Contributor

@KiranSatyaRaj KiranSatyaRaj commented Aug 27, 2022

adding primary service models to the documentation and modifying it for a better understanding and readability.

Signed-off-by: Kiran Satya Raj 90622705+KiranSatyaRaj@users.noreply.github.com

Describe your changes

Related issue number or link (ex: resolves #issue-number)

Checklist before opening this PR (put x in the checkboxes)

  • This PR does not contain plagiarism
    • don’t copy other people’s work unless you are quoting and contributing it to them.
  • I have signed off on all commits
    • signing off (ex: git commit -s) is to affirm that commits comply DCO.

adding primary service models to the documentation and modifying it for a better understanding and readability.

Signed-off-by: Kiran Satya Raj <90622705+KiranSatyaRaj@users.noreply.github.com>
@netlify
Copy link

netlify bot commented Aug 27, 2022

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit 616573c
🔍 Latest deploy log https://app.netlify.com/sites/cncfglossary/deploys/6320464a69f4df00099bd624
😎 Deploy Preview https://deploy-preview-1270--cncfglossary.netlify.app/cloud-computing
📱 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.

@jihoon-seo jihoon-seo added lang/en for English update term Updates to existing terms maintainers Use this label if PR requires maintainers to take action labels Aug 27, 2022
Copy link
Collaborator

@iamNoah1 iamNoah1 left a comment

Choose a reason for hiding this comment

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

Thanks @KiranSatyaRaj for your contribution. I have some change requests :)

content/en/cloud-computing.md Outdated Show resolved Hide resolved
content/en/cloud-computing.md Outdated Show resolved Hide resolved
Copy link
Contributor Author

@KiranSatyaRaj KiranSatyaRaj left a comment

Choose a reason for hiding this comment

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

I agree with the changes required

content/en/cloud-computing.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@iamNoah1 iamNoah1 left a comment

Choose a reason for hiding this comment

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

added actionable suggestions :)

content/en/cloud-computing.md Outdated Show resolved Hide resolved
content/en/cloud-computing.md Outdated Show resolved Hide resolved
Signed-off-by: Noah Ispas <13436814+iamNoah1@users.noreply.github.com>
Signed-off-by: Noah Ispas <13436814+iamNoah1@users.noreply.github.com>
@iamNoah1
Copy link
Collaborator

iamNoah1 commented Sep 6, 2022

thanks @KiranSatyaRaj :)

Signed-off-by: Catherine Paganini <74001907+CathPag@users.noreply.github.com>
Copy link
Collaborator

@CathPag CathPag left a comment

Choose a reason for hiding this comment

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

LGTM

@CathPag CathPag merged commit 1e62098 into cncf:main Sep 13, 2022
@KiranSatyaRaj KiranSatyaRaj deleted the patch-2 branch September 13, 2022 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/en for English maintainers Use this label if PR requires maintainers to take action update term Updates to existing terms
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants