search

chitezh / woocommerce_states_places

Woocommerce plugin that lists states, cities, local government areas and towns
https://wordpress.org/plugins/states-cities-and-places-for-woocommerce
GNU General Public License v3.0
50 stars 141 forks source link

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

Open cquesadad opened 2 years ago

cquesadad commented 2 years ago

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 commented 1 year ago

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.