[Spanish] localize 'Platform as a Service (PaaS)' #1400 #1424

mdf-ido · 2022-10-28T20:14:48Z

Describe your changes

Adding the description of PaaS in spanish.

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. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

netlify · 2022-10-28T20:14:53Z

✅ Deploy Preview for cncfglossary ready!

Name	Link
🔨 Latest commit	`2ed578a`
🔍 Latest deploy log	https://app.netlify.com/sites/cncfglossary/deploys/6362dca901bf5900075a9e96
😎 Deploy Preview	https://deploy-preview-1424--cncfglossary.netlify.app/es/platform-as-a-service
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

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

mdf-ido

Added the opening question marks

electrocucaracha

@mdf-ido muchas gracias por realizar la traducción de este término. He realizado un par de sugerencias necesarias en este cambio. Algo importante a notar es que este PR debe apuntar al branch dev-es . Tambien es necesario que los commits sean firmados para que puedan pasar las validaciones del CI.

content/es/platform-as-a-service.md

mdf-ido · 2022-10-31T17:43:03Z

@electrocucaracha - Pregunta: Sera que mejor hago un pull de dev-es en vez de forking el repo?

electrocucaracha · 2022-10-31T19:13:18Z

@electrocucaracha - Pregunta: Sera que mejor hago un pull de dev-es en vez de forking el repo?

El fork es necesario, el detalle es que cuando creas tu branch local debe ser utilizando dev-es como base.

$ git clone git@github.com:mdf-ido/glossary.git
...
$ cd glossary
$ git switch dev-es
$ git branch 1400-spanish-localize-platform-as-service
$ git switch 1400-spanish-localize-platform-as-service

Y de igual manera, al crear el PR, este debe de ir dirigido hacia dev-es.

Por último, es necesario agregar varias palabras en el diccionario para evitar errores en el CI, estas palabras están ordenadas alfabéticamente para detectar duplicidad fácilmente.

mdf-ido · 2022-10-31T19:44:49Z

@electrocucaracha - Pregunta: Sera que mejor hago un pull de dev-es en vez de forking el repo?

El fork es necesario, el detalle es que cuando creas tu branch local debe ser utilizando dev-es como base.
$ git clone git@github.com:mdf-ido/glossary.git
...
$ cd glossary
$ git switch dev-es
$ git branch 1400-spanish-localize-platform-as-service
$ git switch 1400-spanish-localize-platform-as-service
Y de igual manera, al crear el PR, este debe de ir dirigido hacia dev-es.

Por último, es necesario agregar varias palabras en el diccionario para evitar errores en el CI, estas palabras están ordenadas alfabéticamente para detectar duplicidad fácilmente.

Gracias por su ayuda, las palabras que tengo que añadir al diccionario son las que el Spell check encontró?

electrocucaracha · 2022-10-31T19:56:57Z

Gracias por su ayuda, las palabras que tengo que añadir al diccionario son las que el Spell check encontró?

Asi es, basicamente son:

PaaS
fundamentals
platform
App
Engine
Foundry
Heroku

y luego ordenarlas en orden alfabético

$ sort -f -o content/es/.wordlist.txt content/es/.wordlist.txt

mdf-ido

@electrocucaracha - Ya cambie el branch y actualize el diccionario. Gracias por su ayuda y espero aprender en esta para hacer otras y ayudarles!

electrocucaracha · 2022-11-01T15:55:23Z

@electrocucaracha - Ya cambie el branch y actualize el diccionario. Gracias por su ayuda y espero aprender en esta para hacer otras y ayudarles!

Lo único que no cuadra en este PR es que hay cambios en otros archivos no relacionados, podrías hacer rebase de los cambios?

content/es/.wordlist.txt

mdf-ido · 2022-11-02T03:00:45Z

@electrocucaracha - Vi el último error del spell checker. Crees que necesito añadir la palabra true y asi al diccionario?

content/es/platform-as-a-service.md

electrocucaracha · 2022-11-02T18:01:27Z

@electrocucaracha - Vi el último error del spell checker. Crees que necesito añadir la palabra true y asi al diccionario?

Parece ser que la eliminaste de la lista original.

Por otra parte, quizá sea necesario recrear este PR ya que tienes algunos cambios de main o al menos aparecen otros cambios de archivos no relacionados. Cualquier duda puedes contactarme via slack

* Adding signoff * Adding opening question marks * Update content/es/platform-as-a-service.md Co-authored-by: Victor Morales <chipahuac@hotmail.com> Signed-off-by: moises0077 <moisesdfigueroa@gmail.com>

ramrodo

/lgtm

electrocucaracha · 2022-11-11T19:28:18Z

@seokho-son @jihoon-seo can you help us to merge this PR?

…ncf#1424)" This reverts commit efb5492.

* Update content/es/platform-as-a-service.md Co-authored-by: Victor Morales <chipahuac@hotmail.com> Signed-off-by: moises0077 <moisesdfigueroa@gmail.com>

mdf-ido requested review from raelga, electrocucaracha, krol3 and 92nqb as code owners October 28, 2022 20:14

electrocucaracha added the lang/es for Spanish label Oct 28, 2022

electrocucaracha changed the base branch from main to dev-es October 31, 2022 16:23

electrocucaracha requested review from caniszczyk, CathPag, JasonMorgan, seokho-son, iamNoah1 and jihoon-seo as code owners October 31, 2022 16:23

mdf-ido commented Oct 31, 2022

View reviewed changes

electrocucaracha requested changes Oct 31, 2022

View reviewed changes

mdf-ido requested a review from electrocucaracha October 31, 2022 17:50

mdf-ido commented Nov 1, 2022

View reviewed changes

electrocucaracha reviewed Nov 1, 2022

View reviewed changes

content/es/.wordlist.txt Outdated Show resolved Hide resolved

electrocucaracha reviewed Nov 2, 2022

View reviewed changes

content/es/platform-as-a-service.md Outdated Show resolved Hide resolved

electrocucaracha force-pushed the 1400-spanish-localize-platform-as-service branch 2 times, most recently from 177a71b to e4003f2 Compare November 2, 2022 21:07

[Spanish] localize 'Platform as a Service (PaaS)' cncf#1400

2ed578a

* Adding signoff * Adding opening question marks * Update content/es/platform-as-a-service.md Co-authored-by: Victor Morales <chipahuac@hotmail.com> Signed-off-by: moises0077 <moisesdfigueroa@gmail.com>

electrocucaracha force-pushed the 1400-spanish-localize-platform-as-service branch from e4003f2 to 2ed578a Compare November 2, 2022 21:09

electrocucaracha approved these changes Nov 9, 2022

View reviewed changes

ramrodo approved these changes Nov 11, 2022

View reviewed changes

jihoon-seo removed request for caniszczyk, JasonMorgan, seokho-son, iamNoah1, jihoon-seo and CathPag November 11, 2022 22:01

jihoon-seo approved these changes Nov 11, 2022

View reviewed changes

raelga approved these changes Nov 11, 2022

View reviewed changes

raelga merged commit efb5492 into cncf:dev-es Nov 11, 2022

vasucp1207 added a commit to vasucp1207/glossary that referenced this pull request Dec 5, 2022

Revert "[Spanish] localize 'Platform as a Service (PaaS)' cncf#1400 (c…

cd4808c

…ncf#1424)" This reverts commit efb5492.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Spanish] localize 'Platform as a Service (PaaS)' #1400 #1424

[Spanish] localize 'Platform as a Service (PaaS)' #1400 #1424

mdf-ido commented Oct 28, 2022 •

edited

Loading

netlify bot commented Oct 28, 2022 •

edited

Loading

mdf-ido left a comment

electrocucaracha left a comment •

edited

Loading

mdf-ido commented Oct 31, 2022

electrocucaracha commented Oct 31, 2022 •

edited

Loading

mdf-ido commented Oct 31, 2022

electrocucaracha commented Oct 31, 2022

mdf-ido left a comment •

edited

Loading

electrocucaracha commented Nov 1, 2022

mdf-ido commented Nov 2, 2022

electrocucaracha commented Nov 2, 2022

ramrodo left a comment

electrocucaracha commented Nov 11, 2022

[Spanish] localize 'Platform as a Service (PaaS)' #1400 #1424

[Spanish] localize 'Platform as a Service (PaaS)' #1400 #1424

Conversation

mdf-ido commented Oct 28, 2022 • edited Loading

Describe your changes

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

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

netlify bot commented Oct 28, 2022 • edited Loading

✅ Deploy Preview for cncfglossary ready!

mdf-ido left a comment

Choose a reason for hiding this comment

electrocucaracha left a comment • edited Loading

Choose a reason for hiding this comment

mdf-ido commented Oct 31, 2022

electrocucaracha commented Oct 31, 2022 • edited Loading

mdf-ido commented Oct 31, 2022

electrocucaracha commented Oct 31, 2022

mdf-ido left a comment • edited Loading

Choose a reason for hiding this comment

electrocucaracha commented Nov 1, 2022

mdf-ido commented Nov 2, 2022

electrocucaracha commented Nov 2, 2022

ramrodo left a comment

Choose a reason for hiding this comment

electrocucaracha commented Nov 11, 2022

mdf-ido commented Oct 28, 2022 •

edited

Loading

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

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

netlify bot commented Oct 28, 2022 •

edited

Loading

electrocucaracha left a comment •

edited

Loading

electrocucaracha commented Oct 31, 2022 •

edited

Loading

mdf-ido left a comment •

edited

Loading