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

Update of places in Cuba. Spanish writing correction and some missing… #228

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cquesadad
Copy link

Hi!

Im cuban and I was working in a Woocommerce store and I found out this amazing plugin. Then I realize that there were some mistakes in some cities and it also needed to fix grammar.

I hope you find this solution great and if you want me to contribute more in this matter, pleas just ask.

Best regards

Carlos

@YordanSoares
Copy link
Collaborator

Hola @cquesadad:

¡Gracias por tu contribución!

Mis disculpas por el retraso en la revisión de tu PR. Lamentablemente, he tenido muy poco tiempo disponible los últimos meses, pero planeo reanudar los lanzamientos en las próximas semanas.

Volviendo a tu PR.

Tengo varios comentarios sobre puntos que quisiera que revisáramos antes de poder fusionar tu PR:

  1. He visto que has eliminado La Lisa de La Habana, pero he buscado en Wikipedia y parece que el lugar en realidad existe: https://es.wikipedia.org/wiki/La_Lisa, ¿quizás eliminaste el lugar por error?
  2. Tu código generará errores en su estado actual, ya que eliminaste las comas de separación de los lugares, en algunos arreglos, por ejemplo, aquí.
  3. Has añadido espacios y tabulaciones en varios lugares. Por favor, elimina esto para mantener el código limpio.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants