gophers-latam / challenges

Discord bot for Gophers LATAM
https://gopherbot.onrender.com
4 stars 4 forks source link

Add new commands and enhancements #11

Closed zeroidentidad closed 4 months ago

zeroidentidad commented 4 months ago
zeroidentidad commented 4 months ago

Hola @zeroidentidad! Me gustan muchos los nuevos cambios para el bot, te dejo un par de comentarios con algunas cosas que note:

  1. Hay un bug que cuando ingresas el comando de hours, si el pais no empieza con la letra mayuscula, te muestra cualquier horario. Te paso una imagen del testeo para que lo veas:

image

  1. Estaria bueno mostrar la lista de paises en orden y agregar los paises faltantes de la comunidad de latam. Por que a dia de hoy, se muestran paises que tienen un -3 con el horario ingresado y despues se muestra un +2 por ejemplo del pais siguiente. La lista de paises y su +- de horario lo podemos ver en esta web:

http://www.oia-caci.org/es/paises/time.php

  1. Y por ultimo, agregaria en la descripcion del comando hours que el formato ademas de ser {HH:MM} es formato 24hrs para que se entienda su uso, por que entiendo que no todos los paises usan el mismo formato, por ejemplo hay paises que usan el formato 12hrs.

Sobre el punto 1, considero entonces seria capitalizar el texto del nombre del pais como input

Sobre el punto 2, dado que se una un map mostrar los valores nunca sera en orden posiblemente cambiar a un struct y emplear un array de este, se tomara http://www.oia-caci.org/es/paises/time.php como refencia para agregar el resto de paises y que igual esten soportado en la especificacion https://pkg.go.dev/time#LoadLocation + $GOROOT/lib/time/zoneinfo.zip

Sobre el punto 3, se tratara de hacer mas entendible como ingresar el input de la hora

ThiagoMowszet commented 4 months ago

Hola @zeroidentidad! Me gustan muchos los nuevos cambios para el bot, te dejo un par de comentarios con algunas cosas que note:

  1. Hay un bug que cuando ingresas el comando de hours, si el pais no empieza con la letra mayuscula, te muestra cualquier horario. Te paso una imagen del testeo para que lo veas:

image

  1. Estaria bueno mostrar la lista de paises en orden y agregar los paises faltantes de la comunidad de latam. Por que a dia de hoy, se muestran paises que tienen un -3 con el horario ingresado y despues se muestra un +2 por ejemplo del pais siguiente. La lista de paises y su +- de horario lo podemos ver en esta web:

http://www.oia-caci.org/es/paises/time.php

  1. Y por ultimo, agregaria en la descripcion del comando hours que el formato ademas de ser {HH:MM} es formato 24hrs para que se entienda su uso, por que entiendo que no todos los paises usan el mismo formato, por ejemplo hay paises que usan el formato 12hrs.

Sobre el punto 1, considero entonces seria capitalizar el texto del nombre del pais como input

Sobre el punto 2, dado que es un map mostrar los valores nunca sera en orden posiblemente cambiar a un struct y emplear un array de este, se tomara http://www.oia-caci.org/es/paises/time.php como refencia para agregar el resto de paises y que igual esten soportado en la espeficicacion https://pkg.go.dev/time#LoadLocation + $GOROOT/lib/time/zoneinfo.zip

Sobre el punto 3, se tratara de hacer mas entendible como ingresar el input de la hora

Dale Zero, gracias. Sobre el punto 2, lo que se podria hacer, en vez de tomar el pais de input como referencias podriamos hacer una tabla con los paises y sus +- horas y listarlos de esa manera, es otra alternativa que se me ocurre. Sino podriamos solo listarlos por orden alfabetico y ahi nos ahorramos la comparacion de diferencia horaria ya que solo nos fijariamos en el orden alfabetico de cada pais. Espero que se haya entendido.