Open psiconauta opened 3 years ago
Bueno @GorroRojo, reabro el issue para comentar la implementación. En general joya, pero tiene un par de detalles:
lower
es dependiente de la base de datos. Es un método muy extendido que está tanto en la que usamos para desarrollo como en la de producción, pero idealmente estaría bueno no usar métodos que nos aten a una base de datos en particularfind_by
(que devuelve siempre 1 resultado) por where().first
era innecesario, se podía hacer la misma búsqueda con find_by
downcase
en un username nil
(me toca implementar tests en algún momento)Dejo documentado el cambio acá así no se pierde en el chat, pero eso es lo que habría que hacer básicamente.
@GorroRojo si andás con tiempo, es importante este fix porque cuando le pasás un /contame_de
sin argumentos se rompe. Lo dejé andando en producción pero habría que commitearlo y mergearlo.
En telegram al menos, los usernames son case insensitive